Strip type equations when checking record field types

This resolves https://github.com/Microsoft/visualfsharp/issues/5281 by stripping type equations when checking record field types.

@Pilchie , is this PR ok for approval?

test Ubuntu16.04 Release_default Build please

Approved for 15.8 Preview 5 pending other code reviews.


(PDB-4347) Make fact path gc table temp, not unlogged

Temp tables are unlogged tables[1], not vice versa (as had been
assumed), so change the gc table to a temp table.
Since unlogged tables aren't temp tables, you could see a table name
conflict if, for example, multiple gc attempts ran simultaneously:
WITH recursive live_paths(path, value) AS
...
SELECT path
INTO   unlogged tmp_live_paths
FROM   live_paths
was aborted: ERROR: relation "tmp_live_paths" already exists
...
[1] https://rhaas.blogspot.com/2010/05/global-temporary-and-unlogged-tables.html

Test FAILed

That default travis job likely failed because the job was reaped while it was running, let's see how PR Tests does


Widget issue

Hi,
the widget only show the title but no tasks. If i open the app the tasks are shown corretly. Sometimes the tasks show up in widget, but havent figured out yet when.
I face issue on both phones, cm12.1 and cm13 with Nova Launcher and as I remember Apex stil had.

The widget does only show tasks that are currently active, i.e. that have no start date or a start date that lies in the past, but no tasks with a start date in the future. In the app itself, all tasks are shown independent of their status. Does this correspond to your situation?

Hi,
no, the widget does not show any task at all. I delete the widget,
readd, and works again for a while.
On 2016-02-12 12:21, benedetto wrote:

The widget does only show tasks that are currently active, i.e. that have no start date or a start date that lies in the past, but no tasks with a start date in the future. In the app itself, all tasks are shown independent of their status. Does this correspond to your situation?
--
Reply to this email directly or view it on GitHub [1].

Links:
[1] https://github.com/dmfs/opentasks/issues/202#issuecomment-183284030

Can you reproduce this with a different launcher?

Nova and Apex both failed. I will try with built in trebuchet.
On 2016-02-12 15:31, Marten Gajda wrote:

Can you reproduce this with a different launcher?
--
Reply to this email directly or view it on GitHub [1].

Links:
[1] https://github.com/dmfs/opentasks/issues/202#issuecomment-183352841

Hi,
the same happend with default launcher. :(
Üdvözlettel:
Vukovics Mihály
On 2016.02.12. 15:56, szun wrote:

Nova and Apex both failed. I will try with built in trebuchet.
On 2016-02-12 15:31, Marten Gajda wrote:

Can you reproduce this with a different launcher?
—
Reply to this email directly or view it on GitHub
https://github.com/dmfs/opentasks/issues/202#issuecomment-183352841.

I see. Which widget are you using? The resizable or the fixed size one? Can you try to the other one? Maybe just one of them is buggy.

I used the 4x4 one, now the 2x1, and seems to work!
On February 22, 2016 11:50:18 Marten Gajda<EMAIL_ADDRESS>wrote:

I see. Which widget are you using? The resizable or the fixed size one? Can
you try to the other one? Maybe just one of them is buggy.

Reply to this email directly or view it on GitHub:
https://github.com/dmfs/opentasks/issues/202#issuecomment-187120493

Great! So it's the large widget only. We'll check what's wrong.  Btw, are you on <IP_ADDRESS>? I've fixed an issue like that recently.

Hi,
yes, using the latest <IP_ADDRESS>!
On 2016.02.23. 17:22, Marten Gajda wrote:

Great! So it's the large widget only. We'll check what's wrong. Btw,
are you on <IP_ADDRESS>? I've fixed an issue like that recently.
—
Reply to this email directly or view it on GitHub
https://github.com/dmfs/opentasks/issues/202#issuecomment-187771008.

This should be fixed in the staging branch and with the next release.


ftime is deprecated; not sure what you want to do with this

Build warning:
cc 8086tiny.c -DNO_GRAPHICS -O3 -fsigned-char -std=c99 -o 8086tiny
8086tiny.c: In function 'main':
8086tiny.c:808:49: warning: 'ftime' is deprecated: Use gettimeofday or clock_gettime instead [-Wdeprecated-declarations]
  808 |                                                 ftime(&ms_clock);
      |                                                 ^~~~~
In file included from 8086tiny.c:12:
/usr/include/x86_64-linux-gnu/sys/timeb.h:29:12: note: declared here
   29 | extern int ftime (struct timeb *__timebuf)
      |            ^~~~~

nothing to worry about, just a warning


TQDM: AttibuteError del tqdm_class._lock on async / threaded usage

Describe the bug
I somehow get a race condition, when downloading the model in threads. This happens around 1/100 runs in the CI.
This is pretty much how my code looks like:
from huggingface_hub import snapshot_download
from huggingface_hub.utils import disable_progress_bars
import os
import asyncio

disable_progress_bars()
os.environ.setdefault("HF_HUB_DISABLE_PROGRESS_BARS", "True")

def download_sync(model_name):
      return huggingface_hub.snapshot_download(model_name)

async def download(model_name):
      return await asyncio.to_thread(download_sync, model_name)

async def collect()
      asyncio.gather([download("model1"), download("model2")]

asyncio.run(collect())

Sorry for this ugly stacktrace from GCP Cloud build:
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": mydownloader/downloader.py:50: in async_download
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     self._model_full_path = await asyncio.to_thread(
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/lib/python3.10/asyncio/threads.py:25: in to_thread
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     return await loop.run_in_executor(None, func_call)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/lib/python3.10/concurrent/futures/thread.py:58: in run
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     result = self.fn(*self.args, **self.kwargs)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": mydownloader/downloader.py:72: in _download
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     downloaded_path: str = snapshot_download(
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/local/lib/python3.10/dist-packages/huggingface_hub/utils/_validators.py:118: in _inner_fn
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     return fn(*args, **kwargs)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/local/lib/python3.10/dist-packages/huggingface_hub/_snapshot_download.py:239: in snapshot_download
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     thread_map(
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/local/lib/python3.10/dist-packages/tqdm/contrib/concurrent.py:69: in thread_map
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     return _executor_map(ThreadPoolExecutor, fn, *iterables, **tqdm_kwargs)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/local/lib/python3.10/dist-packages/tqdm/contrib/concurrent.py:47: in _executor_map
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     with ensure_lock(tqdm_class, lock_name=lock_name) as lk:
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": /usr/lib/python3.10/contextlib.py:142: in __exit__
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     next(self.gen)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": tqdm_class = <class 'huggingface_hub.utils.tqdm.tqdm'>, lock_name = ''
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     @contextmanager
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":     def ensure_lock(tqdm_class, lock_name=""):
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         """get (create if necessary) and then restore `tqdm_class`'s lock"""
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         old_lock = getattr(tqdm_class, '_lock', None)  # don't create a new lock
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         lock = old_lock or tqdm_class.get_lock()  # maybe create a new lock
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         lock = getattr(lock, lock_name, lock)  # maybe subtype
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         tqdm_class.set_lock(lock)
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         yield lock
2024-01-19 01:11:22.306 CET
Step #1 - "build-image":         if old_lock is None:
2024-01-19 01:11:22.306 CET
Step #1 - "build-image": >           del tqdm_class._lock
2024-01-19 01:11:22.307 CET
Step #1 - "build-image": E           AttributeError: _lock
2024-01-19 01:11:22.307 CET
Step #1 - "build-image":
2024-01-19 01:11:22.307 CET
Step #1 - "build-image": /usr/local/lib/python3.10/dist-packages/tqdm/contrib/concurrent.py:24: AttributeError
2024-01-19 01:11:22.307 CET
Step #1 - "build-image": ---

Reproduction
No response
Logs
No response
System info
hf-transfer = "0.1.3"
huggingface-hub = "0.17.1"
python = "~3.10.2"

@michaelfeil Thanks for reporting and sorry for the inconvenience. Looks like a known and fixed issue (see https://github.com/huggingface/huggingface_hub/pull/1629). Could you update your huggingface_hub version to a more recent one (latest is 0.20.2). It should solve your issue :)

I'll give it a shot to bump the version.


Radar Chart doesn't always render a value

I have a 4 column radar chart.
It will often not render a column.
For example:
var data = {
	labels : ["A","B","C","D"],
	datasets : [
		{
			fillColor : "rgba(220,220,220,0.5)",
			strokeColor : "rgba(220,220,220,1)",
			pointColor : "rgba(220,220,220,1)",
			pointStrokeColor : "#fff",
			data : [66,66,66,50]
		}
	]
}

The last column will mostly never render.  It may have once.
If I switch the value, it will sometimes render , and sometimes not.  if it does render, it will not always render correctly ( it will appear to be a fraction of the value ).
I tried adding new datapoints to see if order affected this. I have not been able to predict if the new or previously missing datapoints will render or not.
This chart seems fundamentally flawed.

@jvanasco I cannot reproduce this issue with the latest version (1.0-beta4) of Chart.js. Since the time that you reported this issue, the library has undergone a significant rewrite. If your issue still persists, please open a new issue and include a link to a JSBin demonstrating the problem.
This is the JSBin that I used to test the issue.


deprecate breaking-only

Codecov Report

Merging #263 (7a60ef1) into main (94a44bd) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #263   +/-   ##
=======================================
  Coverage   76.79%   76.79%
=======================================
  Files         153      153
  Lines        8574     8574
=======================================
  Hits         6584     6584
  Misses       1655     1655
  Partials      335      335

Flag
Coverage Δ

unittests
76.79% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/flags.go
85.08% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Corrected typo's on en & zh-cn 2022-02-17-updated-dockershim-faq.md files

Updated typo's on the following files (changed incluides to includes):
content/en/blog/_posts/2022-02-17-updated-dockershim-faq.md
content/zh-cn/blog/_posts/2022-02-17-updated-dockershim-faq.md

/assign @onlydole


Improve tokenizer usage in MessageController extract action

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
?

Fixed issues
https://github.com/yiisoft/yii2/issues/16828

Merged. Thank you!


Callback event for swipeout "cancel" in dialog

Hello and thanks for geat work you do here.
Can I get a sample code for the swipeout - cancel event?
Using this code right now to delete products
$('.deleted-callback').on('swipeout:deleted', function() {
    deleteProduct($(this).data('id'));
});

<li class="swipeout deleted-callback" data-id="my-custom-id-here">
...
</div>

When I swipe the popup shows with ok and cancel buttons. If I press cancel, the swipe remains open, need to close it.
Btw, when is the v3 going to be released?
Thanks

Can't see much sense in such event. It is better and easier to do it manually then:
$('.swipeout-my-delete').on('click', (e) => {
  const $swipeoutEl = $clickedEl.closest('.swipeout');
  if ($swipeoutEl.length === 0) return;
  app.dialog.confirm('Are you sure you want to delete this?', 'My App Title',
    () => {
    app.swipeout.delete($swipeoutEl);
  }, () => {
    app.swipeout.close();
  });
});


Self training convenience

Self-Training Convenience
Originally just some things to make it easier for me to get lots of data, but I've tried polishing it.
I deviated from the original code style, but it makes it easier for me to look at (:
Changes:
Self training:

Session time adjustable as a constant in code
Buffer time (Time between hitting enter and it recording) adjustable as a constant in code
New argument overwrite which allows control over saving multiple training sessions

0 is no overwrite, 1 is yes overwrite. Default is 1 (same behavior as before)

Model training:

Can combine all .pkl files in the model\intent directory.

Can combine files with varying amounts of actions (A warning is given when this is done)

Obligatory :3

Would it be possible to instead add the constants as a command line arguments instead?


Access Check Failed error on HEDA Settings. Missing class attribute in CMP_Acc_Naming_Dropdown

Bug
If Lighting debug mode is enabled for a user, that user will see the following error at the bottom of the HEDA Settings page

This page has an error. You might just need to refresh it. Access Check Failed! AttributeSet.get(): attribute 'class' of component 'markup://hed:CMP_Acc_Naming_Dropdown {1150:0}' is not visible to 'markup://aura:if {1152:0}'. Failing descriptor: {aura:if}

The cause is in the CMP_Acc_Naming_Dropdown component. "v.class" is referenced several times in the markup but there is no aura:attribute named "class" defined. The error is getting caught and reported in debug mode, but otherwise you wouldn't see it.

**Lurch: add

Tracking W-032060

Background:

This issue was reported via Salesforce case escalation on March 20th, 2019.
The org in question has both NPSP and HEDA installed.
The issue may be associated with a known issue that was fixed in Spring 19 related to Enabling Debug mode for Lightning components
The issue may be due to a bug in the lighting component code.
@kevinswiggum was able to replicate the error in one of his orgs by turning on Lightning Debug mode. Turning off Debug mode prevents the error.
Recommendation is for the customer to disable debug mode which will remove the error.

Is there any relation to Issue 584?
@aligeez - FYI

@jacebryan  & @kevinswiggum  Thanks for all of the helpful information here. This is not related to #584, but it is definitely similar! Fix coming soon :)

Included in beta release 1.74 (Beta 1)

Included in production release 1.74


Enable F1 for TFM in Visual Studio project files

Summary
The F1 keyword is already emitted, but is not found, so the F1 service takes users to the page for the parent element, PropertyGroup, which is in the MSBuild docs. A better experience is to take users to this page describing the various values of the property.

Internal previews

📄 File
🔗 Preview link

docs/standard/frameworks.md
Target frameworks in SDK-style projects

#sign-off


Change to default volumeBindingMode?

In this page at https://docs.microsoft.com/en-us/azure/aks/azure-disk-csi#create-a-custom-storage-class, it says "The default storage classes use a volumeBindingMode: Immediate class"
I think the default has actually changed to volumeBindingMode: WaitForFirstConsumer per what I'm seeing in new AKS clusters I create:
kubectl get sc
NAME                    PROVISIONER                RECLAIMPOLICY   VOLUMEBINDINGMODE      ALLOWVOLUMEEXPANSION   AGE
azurefile               kubernetes.io/azure-file   Delete          Immediate              true                   28m
azurefile-csi           file.csi.azure.com         Delete          Immediate              true                   28m
azurefile-csi-premium   file.csi.azure.com         Delete          Immediate              true                   28m
azurefile-premium       kubernetes.io/azure-file   Delete          Immediate              true                   28m
default (default)       kubernetes.io/azure-disk   Delete          WaitForFirstConsumer   true                   28m
managed-csi             disk.csi.azure.com         Delete          WaitForFirstConsumer   true                   28m
managed-csi-premium     disk.csi.azure.com         Delete          WaitForFirstConsumer   true                   28m
managed-premium         kubernetes.io/azure-disk   Delete          WaitForFirstConsumer   true                   28m

If the default volumebindingmode has indeed changed to WaitForFirstConsumer, could the page be updated to reflect this?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4ba9a74f-e0b7-5e2c-b066-5f32dc47f950
Version Independent ID: 919e9aef-832f-574f-89b3-0456c5cc3f68
Content: Use Container Storage Interface (CSI) drivers for Azure Disks on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-disk-csi.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: mlearned

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for bringing this to our attention. I have created a PR with the update, changes will be live after author review.
We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Drop voicings

Hi
Please add drop voicings
As you may know, this is a great and systematic way to find useful sets of spread voicing for all plain four-note sixth and seventh chords.
It isn't very fruitful for sixth and seventh chords with extensions or alterations. But for the plain ones - C6, Cm6, C7, Cmaj7, CmMaj7, Cm7, C7b5, and Cdim7 - it's an excellent tool. But I haven't found any app that can generate diagrams for them. So if you manage to do that, it will be a great contribution to the guitar community.
All drop 3 voicings of G7 on string set 0x000x

All of the drop 2 voicings of G7 on string set xx0000

All of the drop 2 voicings of G7 on string set x0000x

All of the drop 2&4 voicings of G7 on string set x00x00

Other string sets:
Drop 2&4: 00x00x.
Drop 3: x0x000.
Drop 2: 0000xx

Hello,
Thanks for your suggestion and detailed examples!
This is indeed one of the evolution I was planning to implement.
I'll get you noticed when it will be available.
Have a nice day,
Modrigue

It's becoming one of the best apps for exploring chords and scales! Impressed how many scales and chords you have managed to add. I thought Impro-Visor had the most, but you've massively surpassed that, even with microtonal chords!
https://github.com/Impro-Visor/Impro-Visor/blob/master/vocab/My.voc
I'm not a programmer, but I notice that for dim7 chords there are no 4 note voicings selectable [MAX]. But I think most guitarists would view the common 4 note variants as most useful. You may know this but.
Drop2
Bdim7: xx0101 | xx3434 | xx6767 |  x-x-9-10-9-10 |
Bdim7: x2313x | x5646x | x8979x | x-11-12-10-12-x |
Bdim7: 1201xx | 4534xx | 7868xx | 10-11-9-10-x-x |

Drop 3
Bdim7: 1x010x | 4x343x | 7x676x |  10-x-9-10-9-x |
Bdim7: x2x131 | x5x464 | x8x797 | x-11-x-10-12-10 |
And you may know that chords are often simplified by omitting chord tones. The commonly leave out the fifth and even the root - relying on the bass player to cover the root.
No5
CM9: x3243x.
G13: 3x3200
G13: 3x3455

NoRno5
G13: xx3200
G13b9: xx3454

Thank you once more for your very constructive and interesting feedback and ideas, as well as your detailed examples!
I already took note and promise to add drop voicings as soon as I'll have sufficient time to implement it the right way :-)
Thanks again,
Have a nice day,
Modrigue


Question more then issue...

Its not super clear if I can persist the location of the drag and dropped items? Like generate the order from a model?
So far super awesome tool

UP 👍

One quite easy way to do this is to cancel all drop events but use the event information to manually update the position of the item in the model.


Git Powered CMS

Full SaaS application running on Cloudflare is great. But are there any plans to build a CMS that commits to git to keep the content in the repo only and build with the CI/CD on Cloudflare?
Also if the website content is stored in the KV and a static JAMSTACK site is built, direct way to access KV from the API in the build phase, and CMS can direct save in KV would be great.

This is more of a static-site generator feature and not something a SaaS would offer. With git-based systems, there's no reason to use anything over the network since the entire content & assets are already available and managed locally.
Your second paragraph is definitely possible and is the intended usage. Any existing jamstack site would be making API requests for its content at build-time. The requests made are determined entirely by the user, so they have full control. The usage will match existing headless CMS platforms (eg, Sanity or Contentful) if you're familiar with them
Closing, but please let me know if I misunderstood anything!

Got your point. I was asking if we could build something that will store data in KV instead of the Sanity... So your CMS would be saving changes in the Workers KV (instead of Sanity) and using KV API to get data at build time

Correct, data is stored in KV.
But at build-time, you'll make HTTP calls for your data from the deployed demo (or your own hosted version). This is common to all headless CMS integrations.

Correct, that's great. Looking forward for the initial version.


[fix] Fixed autocomplete filter exception on invalid input #326

It would be good to send this fix upstream but it seems that the package we depend on is not really active in maintaining it so let's fix it here for now and in the future we'll look for a better solution or fork the package to maintain it on our own.

Coverage increased (+0.02%) to 98.673% when pulling aaa52a132ecc503c63420a6a4b471c4e7f042046 on catch-validation-error-autocomplete into bf1d7d780b249314256e56ea878440daf93ed47f on master.


Crash when used with ES6 syntax

The tern server crashes as soon as I move inside the parentheses of a function call or declaration to inspect the inferred parameter types.
Stack trace:
tern/lib/infer.js:1531 self.propagate(new HasMethodCall(pName, args, node.arguments, out));
                                     ^ TypeError: self.propagate is not a function
  at tern/lib/infer.js:1531:14
  at tern/lib/infer.js:1359:7
  at infer (tern/lib/infer.js:1607:22)
  at Object.exports.inferWrapper.walk.make.Expression (tern/lib/infer.js:1616:7)
  at c (tern/node_modules/acorn/dist/walk.js:68:35)
  at Object.base.ExpressionStatement.base.ParenthesizedExpression (tern/node_modules/acorn/dist/walk.js:186:10)
  at c (tern/node_modules/acorn/dist/walk.js:68:35)
  at Object.exports.inferWrapper.walk.make.Statement (tern/lib/infer.js:1628:7)
  at c (tern/node_modules/acorn/dist/walk.js:68:35)
  at Object.base.Program.base.BlockStatement (tern/node_modules/acorn/dist/walk.js:180:5)

.tern-project file:
{
	"ecmaVersion": 6,
	"libs": [],
	"loadEagerly": [
		"src/**/*.js"
	],
	"plugins": {
		"doc_comment": {
			"fullDocs": true
		},
		"node": {},
		"complete_strings": {}
	}
}

This only happens in larger projects and apparently only when using ES6 syntax. I was not able to quickly put together a small staged example where this happens but I can reliably reproduce this issue when working with this repository. The issue is reproducible at pretty much every js file in there.
I'm using Atom 1.1.0 on OS X 10.11.1 with the latest version of this extension (0.8.4 as of writing) and since tern is included via GitHub I made sure that the currently latest commit https://github.com/ternjs/tern/commit/5e9b6728219ab443a7dd85222f6e06458bb1b5f1 is installed. Idk if it matters but the globally installed node version is 5.0.0 with npm 3.3.6.
It looks like this is a problem with tern itself but since I'm not familiar at all with the codebase I thought it was best to report this issue in a top-down order.
If you need any further information please let me know.

I agree that this looks like a Tern issue. Can you please report it there, too? Thanks.

See https://github.com/ternjs/tern/issues/675. Looks like the issue somehow disappeared after restarting Atom yesterday, which I did before multiple times without success.
I'll report again if the problem should reappear.


start, component and container generator

PR requested by Justin Gordon
created generators in addition to original code. Start generator replaces the HelloWorld folder with an App folder and necessary start files, routes, controller, rails view and .scss file.
Component and Container generators create named component files, insert import from and name into the registration file, rails view, routes, .scss file

This change is

@Luke-Popwell Before I review the code, I'd like to see:

The syntax for calling the generator
public github repo with a PR that shows the effect of running your generator on top of a fresh Rails install with the basic generator all committed.
the same but with the --redux option.

yeah sorry i’ll delete my pull request. sorry to waste your time. no worries best of luck

On Feb 13, 2017, at 12:34 AM, Justin Gordon<EMAIL_ADDRESS>wrote:
@Luke-Popwell https://github.com/Luke-Popwell Before I review the code, I'd like to see:
The syntax for calling the generator
public github repo with a PR that shows the effect of running your generator on top of a fresh Rails install with the basic generator all committed.
the same but with the --redux option.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/shakacode/react_on_rails/pull/714#issuecomment-279305317, or mute the thread https://github.com/notifications/unsubscribe-auth/ANCUG7bto_6g3mG5F7fAkBGWzeNr2Ybrks5rb_mDgaJpZM4L-imf.


enable "add" goal to create new function

@xscript,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #39 into master will increase coverage by 0.31%.
The diff coverage is 83.11%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   80.67%   80.98%   +0.31%
==========================================
  Files          36       39       +3
  Lines        1040     1194     +154
  Branches      110      125      +15
==========================================
+ Hits          839      967     +128
- Misses        163      178      +15
- Partials       38       49      +11

Impacted Files
Coverage Δ

...oft/azure/maven/function/bindings/BaseBinding.java
100% <ø> (ø)
:arrow_up:

...re/maven/function/bindings/StorageBaseBinding.java
100% <ø> (ø)
:arrow_up:

...ft/azure/maven/function/bindings/TableBinding.java
83.33% <ø> (ø)
:arrow_up:

...ft/azure/maven/function/bindings/QueueBinding.java
93.75% <ø> (ø)
:arrow_up:

...oft/azure/maven/function/bindings/BlobBinding.java
95.65% <ø> (ø)
:arrow_up:

...ure/maven/function/bindings/ServiceBusBinding.java
88.63% <ø> (ø)
:arrow_up:

...ft/azure/maven/function/bindings/TimerBinding.java
85.71% <ø> (ø)
:arrow_up:

...azure/maven/function/bindings/EventHubBinding.java
85.71% <ø> (ø)
:arrow_up:

...oft/azure/maven/function/bindings/HttpBinding.java
85.18% <ø> (ø)
:arrow_up:

...com/microsoft/azure/maven/function/DeployMojo.java
83.72% <100%> (+0.38%)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf7f974...6659c2c. Read the comment docs.


Integer Vulnerabilities that Mythril can Detect but Fail to Detect

There is a contract containing three integer vulnerabilities.  The two overflows are at lines 16 and 17 in function mintToken. The underflow is at line 32 in the function burnFrom.
pragma solidity ^0.5.0;
contract Goal {
    address owner;
    uint totalSupply ;
    mapping(address=>uint) balance;
    mapping(address=>mapping(address=>uint)) allowance;

   constructor () public {
          owner = msg.sender;
          totalSupply = 0;
   }

   function mintToken (address target, uint amount)
    public {
         require (msg.sender == owner);
         balance[target] += amount; // overflow
         totalSupply += amount; // overflow
    }

   function approve (address spender, uint value)
    public returns (bool) {
       allowance[msg.sender][spender] = value;
       return true;
    }

   function burnFrom (address from, uint value)
    public returns (bool)
       require (balance[from] >= value);
       require (allowance[from][msg.sender] >= value);
       balance[from] -= value;
       allowance[from][msg.sender] -= value;
       totalSupply -= value; // underflow
       return true;
    }
}

The overflows can be detected if the sequence [mintToken, mintToken] is covered.
Take the example of totalSupply.  Executing the function mintToken, totalSupply has a symbolic value of the type uint, let's say, amount1 (to denote a function parameter, I use the parameter name followed by the position of the function in a sequence).
After executing mintToken again ( i.e., the sequence  [mintToken, mintToken]), totalSupply has the value: amount1+amount2. amount1+amount2 can cause overflow as it can denote a value larger than the maximum value of the type unit.
The underflow can be detected if the sequence[approve, mintToken,  burnFrom] can be covered.
After executing this sequence, totalSupply has the value amount2-value3. Since value3 can have a value larger than amount2, there is an underflow.
Note that the sequence [mintToken, burnFrom] can also make totalSupply has the value amount1 -value2. But in this case, value2 should have value 0 in order to pass the condition at line 29 (require (allowance[from][msg.sender] >= value);. If I am wrong, please correct me). Therefore, amount1-value2 can not cause underflow.
In short, the overflow vulnerabilities can be detected when the depth limit is 2 (or the length of the executed sequences <=2), which is the default case of Mythril. The underflow can be detected when the depth limit is 3.
Let's see how Mythril detects those integer vulnerabilities.
Assume the solc version is 0.5.0. The version of Mythril is v0.23.22. Python has version 3.6.9
myth analyze Goal.sol:Goal --pruning-factor 1.0

By executing the above command, the sequence [mintToken, mintToken] is covered (i.e., is executed and generated a state-changing state), but no vulnerability is detected. The results are in the file: Goal_t2_solver_timeout_default_results.txt
Since whether there is an integer vulnerability or not is determined by the evaluation results of the constraints, it is possible that the constraint solver can not evaluate the constraints in a limited time. So, I increase the timeout of the constraint solver to 1000000. The default timeout is 10000.
myth analyze Goal.sol:Goal --solver-timeout 1000000 --pruning-factor 1.0

As expected, the two overflow vulnerabilities are detected. So, the vulnerability detection is impacted by the timeout of the constraint solver. The results are in the text file:Goal_t2_solver_timeout_1000000_results.txt
However, by increasing the timeout of the constraint solver, the underflow can not be detected when [approve, mintToken, burnFrom] is covered.
myth analyze Goal.sol:Goal -t 3 --solver-timeout 1000000 --pruning-factor 1.0

Here are the results:
Goal_t3_solver_timeout_1000000_results.txt
I am sure that the underflow can be detected. I assign 1 to totalSupply in the constructor. The modified contract is named Goal0. Doing so would make the constraints more simple. The constraints here are the constraints, the satisfaction of which indicates there is an integer bug, while the unsatisfaction of which denotes that there is no integer bug.
Executing the following command, the underflow is detected.
myth analyze Goal0.sol:Goal0 -t 3 --solver-timeout 1000000 --pruning-factor 1.0

Here shows the report of the underflow.

Goal0_t3_solver_timeout_1000000_results.txt
In my opinion, the underflow vulnerability can be detected. I am not clear why totalSupply=1 can have a big impact on the detection of this vulnerability.  Would you explain this observation?

A quick tip, you can execute specific transaction sequences with mythril using:
./myth -v4 a t.sol --solv 0.5.0 -t 3 -m IntegerArithmetics --transaction-sequences [[0x095ea7b3],[0x79c65068],[0x79cc6790]] --disable-dependency-pruning

Just don't forget to disable dependency pruning. You can put a list empty for unconstrained execution for that transaction, or you can add stuff to the list for more functions. Use function signatures from solc --hashes
Coming to underflow
The code has the following check for burnFrom:
require (balance[from] >= value);

Which means value can only be less than balance[from]
You can also observe the following:
mintToken() ->
         balance[target] += amount; // overflow
         totalSupply += amount; // overflow

The totalSupply only increments along with balance. So it cannot underflow without overflowing first (which is not possible in 3 transactions)
In the case when totalSupply is set to 1. We only need 1 transaction to overflow, as uint256_max + 1 will overflow to 0. It's easy to underflow 0. This is not possible in 3 transactions as in above case.
You can check this with the following command:
./myth -v4 a t.sol --solv 0.5.0 -t 4 -m IntegerArithmetics --transaction-sequences [[0x095ea7b3],[0x79c65068],[0x79c65068],[0x79cc6790]] --disable-dependency-pruning

Increase solver timeout if need be.

Indeed, the underflow vulnerability is detected. Here is a screenshot showing the detailed information of this vulnerability.

But I am still not clear about the reason:
case 1: executing the sequence: [[approve],[mintToken],[burnFrom]] can not detect the underflow.
totalSupply =0+amount2-value3
case 2: executing the sequence: [[approve],[mintToken],[mintToken],[burnFrom]] detects the underflow.
totalSupply =0+amount2+amount3-value4
Note that amount2, amount3, value3, and value4 are symbols of type unit.
In case 2, if I understand correctly, amount2+amount3 can cause overflow and thus can have a value 0. Then, 0-value4 is easy to determine that there is an underflow.
On the other hand, in case 1, amount2-value3 is not easy to determine? is the "easy"  means that the constraint solver can easily reason?

 On the other hand, in case 1, amount2-value3 is not easy to determine? is the "easy" means that the constraint solver can easily reason?
that's not the case. As you can see from the code below:
mintToken() ->
         balance[target] += amount; // overflow
         totalSupply += amount; // overflow

A single call to mintToken will be such that balance[target] = totalSupply. As totalSupply and balance[target] are set to 0 initially.
And balance[target] decides value_3:
       require (balance[from] >= value); # "from" would be "target" in previous tx

Which implies it's equivalent to TotalSupply >= value for your tx sequence.

Thanks so much for the detailed explanation. I get your point.


Fps only always shows a black square grid.

That’s stock behavior, this app has no way to modify its appearance.


Application instance workflow

The “dev” instructions in the README are incomplete and inconsistent with the “dev” instructions in eAtlas, which also explains how to use eatlas-template. We should consolidate, and ideally provide an executable example. We should also determine whether there need to be instructions for setting up just an application instance, with no dev capability.

[ ] tgve/eAtlas#94

[ ] move “dev” documentation to eAtlas, link to from eatlas-template

[x] eAtlas README says file: instead of link: in package.json – which is correct?
build automation to verify/replace instructions (see checkout repos side-by-side instructions) – overkill

tgve/eAtlas#106

Checkbox needs to change in order to close this ticket @rolyp.


Updated homepage planet counter

This is a very simple change. I added two text files with the planet and paper counts (the code to generate will be added in a future pull request), and changed the planet_counter html file to use those values instead.

Hi @Nschanche great work on the statistics page. I see you've made some .html pages that are plotly plots. This is great! Let's include the script you use to generate these files in this PR, because otherwise we won't be able to recreate these plots. We also need to have access to the data used to generate the plots, so let's add that in a sensible place too!

LGTM, let's merge!


fix:Repair broken urls and link on the GitHub Pages version

This was fixed with 39ba8f4, 88def88 and af03f1c


[reddit] test of user and submission conversion to activity stream objects

Here is a test of the conversion from praw objects to activitystream objects.  I have written fake classes that contain the attributes needed from the praw models.
I could see that coveralls had dropped by 1% on the last merge.  How can I check if this gets it back up?

I could see that coveralls had dropped by 1% on the last merge. How can I check if this gets it back up?

ooh thanks for looking! you'd click Show all checks above, then Details to get to the CircleCI build, then click on the Artifacts tab, which has the raw HTML coverage output:
https://2109-3557653-gh.circle-artifacts.com/0/tmp/coverage_html/index.html
coveralls itself doesn't run on fork builds, so that its token isn't exposed to untrusted code, but once the PR is merged, you can find the coveralls build URL at the bottom of the Python 3 build step log in CircleCI.


The .json file in the get_corpus.py code was not found

The '/data0/xp/gec/ChineseNMT/data/json/train_nlpcc.json' file cannot be found in the links provided for the three datasets by the author.

I apologize for my mistake. Since the project has been over for a long time, my server account and data have been deleted. Perhaps you can refer to other similar CGEC projects e.g., MuCGEC and NLPCC 2018 official data set to organize this data into the desired format. Good luck :)

Thank you for the response, I will try using the similar projects you provided.


Fix release tag stamping

From: Istio Mixer: version: unknown (build: 2017-09-13-a4a5194, status: Modified)
To: Istio Mixer: version: 0.2.2-4-gee30dc0 (build: 2017-09-13-ee30dc0, status: Clean)
This PR eliminates the --exact-match tag in the git describe command, which was preventing the command from ever returning any results.  This will allow us to more quickly assess the version of Mixer in use.
Release note:

This change is

/lgtm


Error received when executing 'az aks update'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks update Extension Name: aks-preview. Version: 0.5.94.
Errors:
The command failed with an unexpected error. Here is the traceback:
cannot import name 'case_insensitive_dict' from 'azure.core.utils' (/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/core/utils/__init__.py)
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 112, in handler
    client = self.client_factory(self.cli_ctx, command_args) if self.client_factory else None
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/_client_factory.py", line 31, in cf_managed_clusters
    return get_container_service_client(cli_ctx).managed_clusters
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/_container_service_client.py", line 642, in managed_clusters
    from .v2022_07_02_preview.operations import ManagedClustersOperations as OperationClass
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/__init__.py", line 9, in <module>
    from ._container_service_client import ContainerServiceClient
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/_container_service_client.py", line 19, in <module>
    from .operations import AgentPoolsOperations, FleetMembersOperations, FleetsOperations, MaintenanceConfigurationsOperations, ManagedClusterSnapshotsOperations, ManagedClustersOperations, Operations, PrivateEndpointConnectionsOperations, PrivateLinkResourcesOperations, ResolvePrivateLinkServiceIdOperations, SnapshotsOperations, TrustedAccessRoleBindingsOperations, TrustedAccessRolesOperations
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/operations/__init__.py", line 9, in <module>
    from ._operations import Operations
  File "/Users/lcohen/.azure/cliextensions/aks-preview/azext_aks_preview/vendored_sdks/azure_mgmt_preview_aks/v2022_07_02_preview/operations/_operations.py", line 19, in <module>
    from azure.core.utils import case_insensitive_dict
ImportError: cannot import name 'case_insensitive_dict' from 'azure.core.utils' (/usr/local/Cellar/azure-cli/2.37.0/libexec/lib/python3.10/site-packages/azure/core/utils/__init__.py)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks update -n {} -g {} --enable-oidc-issuer --enable-workload-identity

Expected Behavior
Environment Summary
macOS-10.16-x86_64-i386-64bit, Darwin 21.6.0
Python 3.10.4
Installer: HOMEBREW

azure-cli 2.37.0 *

Extensions:
aks-preview 0.5.94

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

@lcohenakam We are looking into it and get back to you for any additional information.

route to CXP team

@lcohenakam can you please try upgrading your CLI to the latest by running az upgrade and try again?

@lcohenakam Please let us know if you still are having problems.

@lcohenakam We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

@lcohenakam can you please try upgrading your CLI to the latest by running az upgrade and try again? You can also try updating your local Homebrew repository information and then upgrade the azure-cli package. brew update && brew upgrade azure-cli

This worked for me, thank you


Adds Mutation Estimated Time to Drawer

Description
Adds an estimated time of migration for the mutate drawer. This is based on a hard-coded equation from Classic. Please reach out and I can send you my sources.
Type of Change

Non breaking change ('update', 'change')

Applicable E2E Tests
N/A

this addresses part of #5048

@Jskobos fixed


Jwt auth service impl test

Created test cases in JwtAuthServiceImplTest.java

Can one of the admins verify this patch?

ok to test


ISPN-9182 Remove compat mode from Tasks and Scripts

https://issues.jboss.org/browse/ISPN-9182

Should be ready now. CI is having some trouble to build due to timeouts, but I run the whole suite locally and it's OK

looks good to me.

integrated! thanks @gustavonalle !


null-safety

Do you plan to update to null-safety ?
Thank you
howing dependencies that are currently not opted in to null-safety.
[✗] indicates versions without null safety support.
[✓] indicates versions opting in to null safety.
Package Name  Current  Upgradable  Resolvable  Latest
direct dependencies:
awesome_card  ✗1.1.0   ✗1.1.0      ✗1.1.0      ✗1.1.0
You are already using the newest resolvable versions listed in the 'Resolvable' column.
Newer versions, listed in 'Latest', may not be mutually compatible.

Closing this as this package is currently in null safety


Small update

Pull Request checklist
Note: these are not necessarily requirements

[ ] I reformatted the code with clang-format
[x] I checked to make sure my submission worked
[x] I am the author of submission or have permission from the original author
[ ] Requires update of the PS2SDK
[ ] Requires update of the gsKit
[ ] Others (please specify below)

Pull Request description
Just a couple small things...
Fix a bug reported by @J013k where mc1 was not being found at boot up.
New apps system now lists Apps on root of memory cards.
Some theme changes were causing issues with themes that omit certain things as reported by @ElPatas1.
Test - https://www.sendspace.com/file/lrwob2

Something is not right with this version.
When I only have MC in 1st slot:

Try to save setting, I'm getting:

After short period of time additionally:

I deleted OPL folder (mc0:/OPL) for this test before saving settings.
When I only have MC in 2nd slot, everything seems to be fine.
I can save setting without any problems.
Also I deleted OPL folder (mc1:/OPL) for this test before saving settings.


Categories for bar charts?

All of the examples seem to only use numeric values for the x-axis on bar charts. This would mean that any categories would need to be series that have their own legend entry. But what about when I want to have a custom text label in the x-axis, and only one series (and therefore only one color)?
I have tried including axis.x.type='category', but it only shows '0' in the x-axis (since my 2D column-oriented array has only one array element at the higher level, i.e., the array index is 0.

@jonlachlan i'm going to jump on your issue here because i'm also having issues with bar charts and categories as well. See my fiddle for a working example of a bar chart with categories.
To see my issues with bar charts and categories, see this fiddle:
http://jsfiddle.net/propagated/1uuj8bdd/
When i call load() with new data it truncates my x-axis rather than add new values to it. If you look at my generate() code, you'll see i have commented out axis.x.type= 'timeseries'.
If you uncomment that and comment out axis.x.type = 'categories' you'll see the behavior on the x-axis that i want, however a timeseries bar-chart looks awful for this data.

Thanks for your working example. I modified to axis.x.type = 'category' and yes it does indeed work. This shows me that I probably have a problem with my data and axis definition.
I see that there's possibly a problem with the timeseries axis. I played around with your code a little, and found an odd behavior: Try changing the first date value in initx from '2013-10-31' to '2013-09-31'. Change it back from 'category' to 'timeseries'. When you run this, the first x axis value will be 10/01/13 upon loading the new data, instead of the expected 10/09/13.

The datetime issue you wrote is the expected behavior. Please try new Date('2013-09-31'). I think you should get Tue Oct 01 2013 09:00:00 GMT+0900 (JST). So, I mean 10/01/13 you get in your code is correct.
Btw, please let me close this issue because it's not a bug and please ask in GoogleGroup first as you can see here https://github.com/masayuki0812/c3#using-the-issue-queue Thanks.


MSFT_SPSearchTopology: Cannot bind argument to parameter 'SearchServiceInstance' because it is null. (FQDN Server Name)

Details of the scenario you tried and the problem that is occurring
I'm trying to set the search topology with FQDN Servernames, so i can use my AllNode Data to configure the topology without having a different Configuration for Stage and Produktion.
While troubleshooting I decleared the servernames directly in the configuration.
As a sanity check i ran with hostnames and the ressource creates a Topology as expected.
Every test started with a new SearchService Application without an topology.
Verbose logs showing the problem
VERBOSE: [ServerHostName]: LCM:  [ End    Set      ]  [[SPSearchTopology]SP-SearchTopology]  in 0.3930 seconds.
PowerShell DSC resource MSFT_SPSearchTopology  failed to execute Set-TargetResource functionality with error message: Cannot bind argument to parameter 'SearchServiceInstance' because it is null.
+ CategoryInfo          : InvalidOperation: (:) [], CimException
+ FullyQualifiedErrorId : ProviderOperationExecutionFailure
+ PSComputerName        : ServerHostName.Domain.tld
Suggested solution to the issue
I did some digging into the ressource code. The handling of FQDN Servername works. The $AllSearchServiceInstances Variable gets filled properly, but i suspect the issue happens on Line 363 inside the MSFT_SPSearchTopology.psm1. In the previous code the FQDN gets converted to a hostname, but not in this part. Unfortunatly my PowerShell is not good enougth for the Build up the topology changes for each object type part and there is a high chance I'm wrong with that.
 # Build up the topology changes for each object type
        @("Admin",
          "Crawler",
          "ContentProcessing",
          "AnalyticsProcessing",
          "QueryProcessing",
"IndexPartition") | ForEach-Object -Process {
....

The DSC configuration that is used to reproduce the issue (as detailed as possible)
SPSearchTopology SP-SearchTopology
            {
                DependsOn = '[SPSearchServiceApp]SP-SearchService'
                PsDscRunAsCredential    = $DSCAccount
                ServiceAppName          = "SearchService_Default"
                Admin                   = @('FQDN#1','FQDN#2')
                Crawler                 = @('FQDN#1','FQDN#2')
                ContentProcessing       = @('FQDN#1','FQDN#2')
                AnalyticsProcessing     = @('FQDN#1','FQDN#2')
                QueryProcessing         = @('FQDN#1','FQDN#2')
                FirstPartitionDirectory = ($ConfigurationData.NonNodeData.SPSearchTopologyIndexPath + '\SearchService_Default\Partition0')
                IndexPartition          = @('FQDN#1','FQDN#2')

The operating system the target node is running
OsName               : Microsoft Windows Server 2016 Datacenter
OsOperatingSystemSKU : DatacenterServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 14393.2724.amd64fre.rs1_release.181231-1751
OsLanguage           : en-US
OsMuiLanguages       : {en-US}

Version of SharePoint that is used (e.g. SharePoint 2016)
SharePoint 2016
Version and build of PowerShell the target node is running
PSVersion 5.1.14393.2636
PSEdition Desktop
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
BuildVersion 10.0.14393.2636
CLRVersion 4.0.30319.42000
WSManStackVersion 3.0
PSRemotingProtocolVersion 2.3
SerializationVersion <IP_ADDRESS>
Version of the DSC module that was used ('dev' if using current dev branch)
<IP_ADDRESS>

We are seeing this as well, (using version 2.5 of the resource) we need to use FQDN in our environment configs because the user accounts we use to compile the config are on a different domain than the servers.

In version 2.5 of the resource, the issue is caused by the presence of the FQDN in the $ComponentsToAdd and the $ComponentsToRemove arrays.  These arrays are populated directly from the function input parameter.  The issue can be resolved by adding the following to the Set-TargetResource method.

Create the $domain variable somewhere around line 229 (inside the Invoke-SPDSCCommand script block).  Variable should be populated using the exact code that is used to populate it in the Get method.
Immediately prior to the foreach loop based on $ComponentsToAdd (~line 377), add the following line:  $ComponentsToAdd = $ComponentsToAdd -Replace ".$domain"
Immediately prior to the foreach loop based on $ComponentsToRemove (~line 418), add the following line:  $ComponentsToRemove = $ComponentsToRemove -Replace ".$domain"

The code in my previous comment will fix the issue with the Set method, but the next issue that occurs when using FQDN is with the Test method.  In the resource's current implementation, the Test method will always fail when using the FQDN because the Get method returns the NetBIOS name.  For my current customer, I went back to the original version of this resource and addressed the FQDN vs NetBIOS name issue in the configuration script (ie - the node data file uses FQDN everywhere except in the Search Topology section and the configuration script has logic to handle translations in instances where nodename needs to be compared to topology elements, such as for identifying the first index server).

First identified bug on line 401 contains two $ signs and does not map to the correct value.

With the code change in place for fixing SearchServiceInstance, the null reference exception was fixed and a new error is presented:

Finally got around to investigate this issue. The issue is located on multiple places:

The Get method always returns NetBIOS names, but the Test methods compares the desired and current state 1-on-1, which will always result in false being returned from the Test method when using FQDNs in the desired config.
As specified by @wfeaser: During population of the ComponentsToAdd and ComponentsToRemove variables, domains are not taken into account. When the desired state uses FQDN, the config will try to add the FQDN to the topology and will try to remove the NetBIOS name from the topology, which is not present in the topology.

I am implementing fixes to both issues, but currently I do not have an environment which is using FQDNs which I can test this on. Does anyone have an environment available that is using FQDNs which can test the solution for me?


noResult does not work

noResult slot does not be displayed before API call in Asynchronous select. there is any solution to display it In the first moment?

Hey!
This was an intentional change as we don’t want to mislead the user saying that there are no results if those results are still being fetched.
So far the only solution I think of is extending the component and providing a custom template (which can be a 1:1 copy of what can be found in /src/Multiselect.vue) with a small modification near the noResults slot.
I hope that helps :)

The problem is that users get confused when nothing is showed while pressing the arrow key on the dropdown list without searching anything. I see they expect to see something by pressing the arrow key and they press it multiple time till I need to tell them that they should not expect anything without searching anything. So I think it would be good idea to no show the arrow key or at least display a text that display "you need to enter search phrase"

@marjandejyab you can use "placeholder" attribute to display your "type to search" message. https://jsfiddle.net/pasierb/uqkkLoc6/2/

For anyone that ends up here, had the same problem as @marjandejyab and ended up doing a weird hack.
I do understand the point of "if user didn't search, this doesnt make sense" but in my case i really needed this.
<multiselect  :class="(selectOptions.length == 0 ? 'emptySelect' :'') " ....
  No results were found.
</multiselect>

...


[components] Create custom search bar

In Meshery UI, we have created a search bar, but it is a mixture of Material UI v4 and MUI v5.
Here is the file for reference: https://github.com/meshery/meshery/blob/master/ui/components/searchcommon.js
There is a need to create a custom search bar in the components package to be reused.
At this time, there is not a default ThemeProvider in this repo, so at this, it would be great to create a custom search bar from the implementation in Meshery UI and display this in the Storybook for now.

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

Hey, can I work on this

This is done


WIP - Add methods to plot the deflected shape

Related to Issue #569.
After running the method .run_unbalance_response() the user is able to call a .plot() method to display the amplitude and phase vs frequency response.
This PR adds an alternative method to plot a 3D orbit (rotor deflected shape) response for a given unbalance: .plot_deflected_shape().
>>> rotor = rotor_example()
>>> speed_range = np.linspace(0, 1000, 101)
>>> response = rotor.run_unbalance_response(node=3,
...                                         magnitude=10.0,
...                                         phase=0.0,
...                                         frequency_range=speed_range)
# plot unbalance response:
>>> fig = response.plot(dof=13)
# plot deflected shape:
>>> speed = 900
>>> fig = response.plot_deflected_shape(speed)

The image below is the current output:

Notice that there's still an empty space. There'll be located the bending moment diagram, which I did not finish yet.


Update README.md

Changing small typo trun to be turn

Coverage remained the same at 100.0% when pulling 404882e7c49e880c0af57655d17aa87cd8713ed5 on apaganobeleno:patch-1 into d902822d4cc09742e326960c813fb7f7ed324fcc on appleboy:master.

@apaganobeleno Thanks  👍


feat: Add LSP server WIP

Signed-off-by: Ce Gao<EMAIL_ADDRESS>

I am closing it and filing a new one with my personal fork branch.


add stamping capability to pdfkit

add e.g. headline, page-numbers or other text/info to existing pdf's,
at coordinates to be specified:
For stamping it would be useful to:

open an existing pdf file
get a specific page within this file
add text to this page at specified coords

similar to https://bitbucket.org/arc42/pdfstamper, but without the dependeny on the Java ecosystem :-)
thx,
Gernot

Editing an existing PDF is a long-requested feature (see #83 and #227 for example). Unfortunately, it will likely need to be a separate project as I don't want to integrate an entire PDF reader into PDFKit. There has been some discussion in those issues, however, from other people interested in doing something like you're describing, so check that out.


Update registry.yaml

Addsa HTML SP to create a links.dev for the meetup

@esau-morais I've just forked the my links repo to the organization page, it contains the page.json file https://github.com/HTML-SP/my-links

@esau-morais I've just forked the my links repo to the organization page, it contains the page.json file https://github.com/HTML-SP/my-links

If you take a look closely, you've forked but didn't meet the expectations described here

@esau-morais oh ok, didn't know about this check, i've just edited the file with the meetup social links. Also, I've seen that I cannot register multiple usernames, but as I'm doing it as an meetup, I hope it is ok

@esau-morais oh ok, didn't know about this check, i've just edited the file with the meetup social links. Also, I've seen that I cannot register multiple usernames, but as I'm doing it as an meetup, I hope it is ok

The CI still failing and now it is returning undefined because there's probably a syntax error on your page.json


Discussion: Package Layout

Determine how we should layout the classes, functions and packages.
Some options:

Everything accessible from timefold.solver
Everything accessible from a package at most 1 level deep to timefold.solver
Copy Java's package layout

Fixed by https://github.com/TimefoldAI/timefold-solver-python/pull/44


Use correct number of trace randomizers in low-degree extension

Per consistency check of the non-linear codeword, the verifier queries each polynomial (in value form, i.e., as a codeword) that goes into the non-linear in one place. Each check reduces the degree of freedom for the queried polynomial. To achieve zero-knowledge, the number of trace randomizers needs to be at least the number of non-linear combination checks.
As per #33, the number of non-linear combination checks needs to be at least 2, which is a low upper bound given the above.
Increase the number of trace randomizers.

As @aszepieniec has pointed out, the verifier gets 2 queries per non-linear combination check because of transition constraints. Thus, the number of trace randomizers needs to be at least twice the number of non-linear combination checks.

Also worth noting is that all traces get randomized, whether it's that of base tables or of extension tables.


Refresh the API

Well, this was not really a exciting task, but someone should do that...
I just added a bunch of methods to the C API, set the initial state of the coupled bodies (so calling to MoorDyn_GetBodyState() before MoorDyn_Init() is giving back the correct result) and updated the wrappers.
Tomorrow I am getting a new baby, so I will be unresponsive for a while. @RyanDavies19 feel free to merge/change whatever you need

P.S. This should fix #193

@sanguinariojoe Congratulations on the new baby! I will review these and merge

OK, I am merging this and let see if on the next release someone complains about something

@sanguinariojoe I'm not entirely sure where things changed here but the fairlead output channel values are now different. See the attached input file as an example, and the image comparing outputs to MDF. I suspect it has to do with changing the getNodeTen function. I looked around a bit and changing the fairlead output written (at line 539 in Line.cpp) to Fnet[N].norm() still causes a difference, which seems like then the value of Fnet at the fairlead node has changed. I don't have time at the moment to find a fix for this, but it will need to be fixed at some point because the difference is not negligible. In the future it might be nice to have some regression tests set up to avoid this, or a github action that tests against MDF (which has regression tests).
Commit 08ed0e089cb0ddea9cc904c01db6ecd1738e8ab3 and earlier does not have this problem, MDC and MDF agreed exactly on values. Something in the changes in this PR broke the fairlead output values.

case1.txt

Yeah, the former MoorDyn_GetLineNodeTen() and moordyn::Line::getNodeTen() has been renamed as MoorDyn_GetLineNodeForce() and moordyn::Line::getNodeForce().
Now MoorDyn_GetLineNodeTen() and moordyn::Line::getNodeTen() are giving the so called wall tension (axial stiffness plus axial damping).
That is consistent with some other instances, as well as some other codes in the wild.
The problem is that I have not updated the output channels.


LD40 Unfinished Games

Hey,
i was wondering if you could include Unfinished games as a filter option, next to Compo | JAM | Both?
Best regards,
Chris

To be done:

[ ] Support Unfinished games (they are currently stored as Jam)
[ ] Replace the [Compo|Jam|Both] filters with [Compo|Jam|Unfinished|All]

Three filter checkboxes would be nice for Compo, All and Unfinished.
The unfinished games are currently on top with the Both filter, because nobody is rating them. They are not included in the All filter on the official LD games page.
I can't find my game at all https://ldjam.com/events/ludum-dare/40/do-not-feed. :(


Release 4.3.0

We'll release fluentd 1.14.3 before releasing td-agent 4.3.0 to merge following in_tail fixes:

https://github.com/fluent/fluentd/pull/3541
https://github.com/fluent/fluentd/pull/3542

FYI: chef is already released win32-service v2.3.0 or later:
https://github.com/chef/win32-service/commit/bde5569a9512f2f7ae07a754ea29f4c7b7b1abd0

Now win32-service gem v2.3.2 is out in rubygems!
https://rubygems.org/gems/win32-service/versions/2.3.2

TODO: Cannot upgrade td due to https://github.com/treasure-data/td/pull/252

We'll publish release announcement for this release at https://www.fluentd.org/blog/
In this announcement, we'll also announce drop schedule:

td-agent 3 is no longer supported, it uses Ruby 2.4 and it's already EOL
td-agent 4 for xenial is no longer supported, because standard support of xenial is already ended.

We'll recommend enterprise support for such old versions if users request it.
Do you have any comments? @cosmo0920 @repeatedly

I have no concerns for this comment.


Huion tablet pressure do not work

Hi,
first - thank you all for such amazing work on wlroots/sway.
I am using current master of wlroots (with -DWLR_USE_UNSTABLE=true) and sway.
My tablet is HUION which works:
Input device: HID 256c:006e Pen
  Type: Tablet tool
  Identifier: 9580:110:HID_256c:006e_Pen
  Product ID: 110
  Vendor ID: 9580
  Libinput Send Events: enabled

But in Krita and Gimp pressure is not working. Is that expected at current stage or is that a problem with my environment. It is working fine under i3/Xorg.
And is there a way to setup tablet buttons? In xorg tablet occurred as separate device (Pad) so buttons can be setup by xsetwacom, which obviously can not work under Wayland.

Sway has only basic drawing tablet support.

Related sway issue: https://github.com/swaywm/sway/issues/2288


requests.exceptions.ProxyError: HTTPSConnectionPool(host='openaipublic.blob.core.windows.net', port=443)

This error is caused by my networking. But I explicitly set LLM_PROVIDER="openai" and MODEL_NAME="gpt-4" in .env file. Why does it still request from Azure API?

Strange. I am unable to reproduce this. This works for me:
LLM_PROVIDER="openai"
MODEL_NAME="gpt-4"
OPENAI_API_KEY="secret-key"

Now it works as expected. I don't why. Let it go.


vcpkg fixes: separate vcpkg-tool and allow manifest mode to work

Description of changes
This MR separate vcpkg-tool so that vpckg always run the wrapper script. This enables manifest mode to work correctly.
Things done
Nothing ;-;
This is a MR on your MR, I'm not so good with nix and I don't know how to test packages from a local nixpkgs rapidly.
I have a nix shell file that called a local package nix file, and this is how I tested.

If you are already using flakes I use this flake to have a dev-environment as default-package which I can enter with nix shell:
{
  inputs = {
    flake-utils.url = "github:numtide/flake-utils";
    nixpkgs.url = "git+file:/home/XXX/nixos-unstable/?shallow=1";
  };

  outputs = {
    self,
    nixpkgs,
    flake-utils,
  }:
    flake-utils.lib.eachDefaultSystem
    (
      system: let
        pkgs = import nixpkgs {inherit system;};
      in {
        formatter = pkgs.alejandra;
        packages.default = pkgs.buildEnv {
          name = "vcpkg-dev";
          paths = with pkgs; [
            vcpkg
            # more packages for doing something with vcpkg e.g. building another package
          ];
        };
      }
    );
}

If you are already using flakes I use this flake to have a dev-environment as default-package which I can enter with nix shell:

I haven't tried flakes since it was experimental and channels seemed to be the official way to use nix. I'm gonna try to enable experimented features to be able to test it outside of a shell.nix file.
I'm still beginner in nix and this is my very first port so be gentle :)

@SebTM I updated the PR, it should be better now, thanks for your suggestions.
Let me know what would be needed to be worthy of merging!

LGTM :)

I'm still beginner in nix and this is my very first port so be gentle :)
I hope it didn't came across otherwise :v:

Let me know what would be needed to be worthy of merging!
You will need to PR against https://github.com/NixOS/nixpkgs/ not this fork - then we need more feedback I would say what is required as I'm not fully certain either regards reproducibility/purity of the build - we might need some kind of dependency hash.

I personally have no merge capability at nixpkgs but when everything seems ready and we have a few approvals we can post it in discourse as well for review/merge 👌🏻

I hope it didn't came across otherwise v

Not at all! All my interactions with the nix community was very welcoming. In fact all your comments were really helpful

You will need to PR against https://github.com/NixOS/nixpkgs/ not this fork

The idea was for @Guekka to merge it then merge the updated PR into nixpkgs. I can just open another one too if that's the right way to do it.

Good to hear :) As it is my first time also being involved in packaging a third-party package-manager I'm not certain but I guess even our state here is not really good-enough to get merged so we will have to ask for further feedback so it make sense to create a PR with your changes as well upstream and explain the circumstances how it came to it ✌🏻 and link them :)


Update the build / nuget pack

Remove the .pp file
Add netstandard1.0 nuget
Update the README

I think the whole nuget spec should be rewritten - right now the format and structure of the code is of the old style NuGet packaging with nuget.exe, etc. all bundled, but since nuget 3.0 this is not needed.
The other targets (win, uap, etc.) all should be updated to use the latest MvvmCross 6.0 reference, and I think the PCL target could also be dropped - MvvmCross 6 transitions to the .Net Standard style of stuff, and I believe this project should follow it too.
Right now I don't have access to a machine with Visual Studio so I can't really make the needed changes, but if this isn't merged by next weekend I'll take a look at it.

Thanks :) I would also like ot migrate all build script to cake....sometime in the future ;)

@fonix232 There is a mac build script:
xamarin-bluetooth-le\.build\build.sh

@flyingxu I doubt I can run any of the build scripts on my Note8 :) Unfortunately I'm temporarily without computer.


Open footer social links in the new tab

Now discord link in the footer doesn't work. It happens because Gatsby.js don't know anything about /chat redirect in the server.js. I suggest open these links in the new tab as it is for the same links in the header.
But also I think that we should duplicate redirects on front side and I have plan to do this in the merging PR.

But also I think that we should duplicate redirects on front side and I have plan to do this in the merging PR.

@pavelgrinchenko could you elaborate on this?


Manual update to lcps_ic.csv

Numbers from 2020-09-23 and 2020-09-22 are missing. The news items have unusual titels / texts so the automatic scraper failed. This pull request adds the missing data manually.

Annoying... But thanks. Can you send me an email? I would like to give you write access such that you can make little changes like this without my intervention.


Add empty dummy data setup

The empty file content is pretty much the same as seeds.exs.
And by running it in our CI script, we can make sure that our dummy data scripts will always be up-to-date 🎉

🦀 Requesting reviewers for this pull request:

@simonprev (contributor with 5 commits in the last 90 days and 15 commits overall)


Variable substitution and active target

Operating System: Windows
CMake Version: 3.16.4
VSCode Version: 1.43.0-insider
CMake Tools Extension Version: 1.3.0
Compiler/Toolchain: g++ cross-compiler

I have a workspace that contains 3 CMakeLists.txt files.  Two  CMakeLists.txt files specify two separate executables and one is a library, common to both.  So I have two targets.  I can select the active target by clicking the Build button in the status bar.  But I want to specify a separate build directory for each target.  I can specify such a directory using "cmake.buildDirectory". Currently I have:
"cmake.buildDirectory": "${workspaceFolder}/source/TargetA_cmake/build_TargetA_${buildType}"
My question is how can I specify a separate build directory for each target, i.e. TargetA and TargetB?
I guess I would achieve that using variable substitution. But which variable would I use?
Is the intention to use the ${variant:VARIANTNAME} syntax?  This doesn't feel right as I want to specify a target name not a variant name.

Please give us more details about your project and what you would like to accomplish. In general, a different build folder is required for different variants (debug/release/etc..), or for different kits, not as much for different targets which usually are built in the same place.
Maybe you mean a separate sub-directory for each target, but within the same build folder?
One build folder is set during the configure operation. Currently we don't support setting more than one build folders during the process of configuring.

@andreeis Thanks for your reply. I have reworked my project to use one build directory for both targets that it produces and it seems to work well.


replace getTables() with getSimpleTableSegments()

ref #4885 .
Changes proposed in this pull request:

replace getTables() with getSimpleTableSegments()

Codecov Report

Merging #5056 into master will not change coverage by %.
The diff coverage is 0.00%.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7846b59...1c0a43d. Read the comment docs.

Pull Request Test Coverage Report for Build 10812

0 of 11 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.117%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/sql92/visitor/impl/SQL92DMLVisitor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/dml/TableFactorSegment.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/statement/dml/SelectStatement.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/impl/MySQLDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/impl/OracleDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/postgresql/visitor/impl/PostgreSQLDMLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/sqlserver/visitor/impl/SQLServerDMLVisitor.java
0
2
0.0%

Totals

Change from base Build 10809:
0.0%

Covered Lines:
11705

Relevant Lines:
20493

💛  - Coveralls

@jingshanglu I found you are familiar with PR rules gradually. It is preferred to remove columns and fix it later. I like it,  clever boy, aha.


Search should search on all the filtered views

People is becoming a bit crazy on the tests when trying to search a dataset on the data library when having the Your datasets filter activated. We should add the small icon we designed (@carlostallon) to the results that come from the data library.
This applies to the create modal window and the datasets dashboard

In short, we want a search that could find the "pattern" in any type (your datasets/maps, shared, data library, ...), right?

yes but we will never mix datasets and maps

Yeah, of course.

Unfortunately, we can't search within data-library and user datasets because the type param is different. For data-library we use remote and for standard datasets we use table.
@juanignaciosl @javisantana @Kartones

Well, we could accept a comma separated list of types and then use it wit an OR, isn't it? There's one single table underneath (visualizations) so it should be easy.

Sounds a good solution to me

Should I open a new ticket @juanignaciosl ?

Since I suppose you'll work in a branch, do the frontend part and then pass me the branch if you want.

branch new-dashboard-search

pushed right now, sorry buddy.

Please confirm me if UI is already sending the comma-separated list of types and then assign this to me.

Ready for testing in staging :)


Add an option for erosions to have different ocean levels

Currently the biome/ocean/level configuration lets you define a Y-value at which water can be generated. This is a single value and applies to the whole erosion.
By adding an optional noise-equation (or any other implementation) this could be further improved (e.g. for making realistic rivers, see attached image).

example

Like I was saying in Discord, this will most likely be implemented as an optional alternative to biome.ocean.level, rather than a full replacement.
Specific implementation ideas:

Add an OceanInterpolator that either takes the constant level, or the interpolated level, and interpolates between samples & biomes
Ensure that interpolation distance is customizable, especially inter-biome distance.

Like I was saying in Discord, this will most likely be implemented as an optional alternative to biome.ocean.level, rather than a full replacement.
Specific implementation ideas:

Add an OceanInterpolator that either takes the constant level, or the interpolated level, and interpolates between samples & biomes
Ensure that interpolation distance is customizable, especially inter-biome distance.


1.19.1/2: Origins 1.7.0 causes Crash with Dehydration Mod

Latest version Crashes with Dehydration Mod installed.
No Crash Report but the Crash does show in the log here:
latest.log

This is an issue on Dehydration's side due to it mixing in to a non-existent method in Origins' /origin command due to a refactor on Origins' side (oops). This is also the case for EnvironmentZ

This should now be fixed on the latest release of Dehydration and EnvironmentZ, so feel free to close this issue Apace :D


Is there a way to stop console from flashing different colors?

Basically, when it completes a build console flashes green, if successful, or red for a second. Is there a way to remove this effect?

You could try to add something like this to your custom CSS:
.build.success,
.build.error {
  animation: none;
}

Thanks!


Add split.js to "rake js". #99

Hi, I added split.js to Rakefile for https://github.com/lautis/uglifier/issues/99 .
I used a patch file.
About the directory name "patches" for the patch file, I referred nokogiri's one.
https://github.com/sparklemotion/nokogiri/tree/v1.6.8.rc3/patches/libxslt
Could you could it?
Thanks.

How?

Hello, are you there?

Sorry for totally forgetting about this.


Sponsors images are not scaled properly

I would like to work on this issue under GSSOC'23. I would appreciate if the issue is assigned to me. The images of the sponsoring companies are blurred, I can fix this issue.

Please submit a pull request for the following. We would be happy to merge it :)

Hello, I'm a contributor to GSSOC'23 I am very much Interested in contributing to this project .I am Skilled in web development and I am ready to collaborate and follow all project guidelines.
I will try to make it as different as possible and exciting also.
Please assign it to me. Thank you in advance.😇

Hello, I am a contributor to GGSoC'23, and I would like to contribute to this issue, I have a good hold on front end development and have done projects earlier as well, please assign me this issue.

I have submitted the pull request. You can review and merge it.


ENH: theme & icons

This PR follows https://github.com/mne-tools/mne-python/pull/10456#issuecomment-1086318010 and updates Qt icons so that they adapt to the theme.

light
dark

And so that we are PyQt6 / non-resource.py compatible, right?

And so that we are PyQt6 / non-resource.py compatible, right?

Indeed, it's a step forward because PyQt6 dropped the Qt resource system.

With the new icon overview_bar:

light
dark

Beautiful!

Thanks @GuillaumeFavelier !


Chapter 9 - Import Card doesn't exist

In Chapter 9 - Streaming, we're tasked to update /dashboard/page.tsx
In the code snippet area we have import Card from '@/app/ui/dashboard/card'; However, this file is never given.

Files in the UI Dashboard from Chapters 1-9:

It's a named export in that file, the course is incorrect. Use:
import { Card } from '@/app/ui/dashboard/cards';

It is a bit confusing because the next part of the course involves using the Cards component, which groups all 4 together, and is the default export.

Yes, the import is wrong. It should be /cards not /card, will push a fix.
@Woozl  To make it less confusing, I'll rename  to  to indicate that it wraps cards.


6 test cases fail in main

3 CborTests and 3 XmlTests fail at the tip of main (99068c0e962dc3221806806b81e5d6320328ef14).

Which ones? I don't see something failing in CI.

Interesting. Guessing it's an issue with your TZ specifically. Do you know what inputs are causing this specifically?

What is my "TZ"? Oh! time zone perhaps? Have you tried running these in Visual Studio? Just the first of these fails in 7 out of 447 test cases.
The first failing test case is:

TypeShape.Tests.CborTests_Reflection.Roundtrip_Value(testCase: TestCase { Value = 12/31/9999 11:59:59 PM, AdditionalValues = System.DateTime[], ExpectedEncoding = , IsStack = False, IsLossyRoundtrip = False, HasRefConstructorParameters = False, HasOutConstructorParameters = False, UsesSpanConstructor = False, Type = System.DateTime, IsNullable = False, IsEquatable = True, IsTuple = False, IsLongTuple = False, IsMultiDimensionalArray = False, IsAbstract = False })

I've tried looking at the test cases but there appears to be several levels of indirection in their generation so I haven't tracked down how these values are originally created. But ya, maybe they work on one time zone and not another.

If I were to make a guess, it's because the test data is using DateTime.MaxValue which fails to roundtrip when the local timezone has a negative offset.


Clear default maskDefinitions

How do I change maskDefinitions for a single item?
If i write:

uiMaskConfig.maskDefinitions({'?': /0-9]/});

result mask

'9': /\d/,
'?': /\d/,
'A': /[a-zA-Z]/,
'*': /[a-zA-Z0-9]/

@Sir-J The readme includes explanation about how to customize the maskDefinitions for a single element and globally. If that doesn't help, then I need more information about what you are trying to do in order to help solve your issue.
Also, using a ? as an character in mask definitions will most likely cause issues as the ? is interpreted as the optional mask character flag.

elatively? I understood you. Tomorrow I'll try to adjust to a different character.
Today, I am doing so:
In control:
$scope,phone = "";
$scope.uiMaskConfig = {
        maskDefinitions({'?': /0-9]/});
};

In view:
<input ui-mask="+7 (9??) ???-??-??" ui-options="uiMaskConfig" ng-model="phone">
After start app i see:

If I comment out this line in iu-utils.js:
angular.extend(current[i], original[i]);
ui-mask working correct.

Ah okay, well it looks like you are using a very old version of ui-mask if it is still in ui-utils.js which might be part of the problem.
If you want to redefine the entire maskDefinitions on an individual element, then you would need to set each key from the defaults to undefined and then define the character you want.
Example:
maskDefinitions: {
      '9': undefined,
      'A': undefined,
      '*': undefined,
      '[YOUR_CHAR]': /[0-9]/
}

However, I am not sure if that will work with your version of the ui-mask library. I would recommend getting the latest version first.

Thank you, I will do so today.


Switch to rasterio

With the addition of the new rasterio based source module in large-image, I have decided to go all in on rasterio here and drop GDAL use. Additionally, I have decided to limit the scope of this project to only geospatial images moving forward (since large-image has development underway for better non-geo visualization in Jupyter).
This is a start to a handful of changes that will ultimately make localtileserver easier to use and more performant.
With this switch to rasterio and some significant internal refactoring, this will warrant a major version bump -- v1.0.0 is on its way

Codecov Report

Merging #154 (c965937) into main (d5bfa56) will increase coverage by 1.27%.
The diff coverage is 88.88%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #154      +/-   ##
==========================================
+ Coverage   85.66%   86.93%   +1.27%
==========================================
  Files          24       24
  Lines        1381     1370      -11
==========================================
+ Hits         1183     1191       +8
+ Misses        198      179      -19

cc @giswqs, I think you'll be happy to see this! But this PR is completely changing the dependency stack and may impact your usage

This is great. It will make the installation much easier. I will keep an eye on this and make changes downstream accordingly

Going ahead with a merge as I'd like to do some other internal refactoring. Release is not coming immediately as I need to do more testing and want feedback from @giswqs

Sure. I can help with testing


Package ID Prefix Issue

I am currently working on a small Markdig extension that adds markdown syntax highlighting by leveraging ColorCode-Universal, which is an updated version of ColorCode.
Unfortunately, I am running into an issue when trying to publish the library as a NuGet package:
error: Response status code does not indicate success: 409 (This package ID has been reserved. Please request access to upload to this reserved namespace from the owner of the reserved prefix, or re-upload the package with a different ID. Go to https://docs.microsoft.com/en-us/nuget/reference/id-prefix-reservation learn more about Package ID prefix reservation.).

I believe the issue is that "Markdig" is a reserved package ID prefix. Removing "Markdig" from the extension name will obscure its purpose from potential users of the library. Is there a path forward to being unblocked and publishing this extension as Markdig.ColorCode?

Is there a path forward to being unblocked and publishing this extension as Markdig.ColorCode?

No there is no path apart prefixing your package with your own name as everybody else. There is one exception for Markdig.Wpf, but this package existed before the Markdig prefix was locked. Having a package like Markdig.ColorCode or any other packages would be very confusing as folks might think that it comes from this repo or it is supported in similar way.

Hey, thanks for your response and clarification. I'll find another package name that works and is still clear about what it does. I'll also be sure to adjust my repository name to match. 😄


format all swift sources, enable format checks

One of those "it has to be done someday" tasks... enabling formatting checking and formatting all Swift sources.
This is just the swift format Sources --recursive --in-place invocation.
We can tweak the config if we don't like the output but it'll be nice to have consistent formatting.
WDYT?

Uncovered fun bug in swift-format: swift-format breaks compilation by wrongly reformatting nested multi-line string blocks https://github.com/swiftlang/swift-format/issues/864