Interval Literal output inconsistent date_type

Describe the bug
A clear and concise description of what the bug is.
the output of INTERVAL XXX is inconsistent
To Reproduce
Steps to reproduce the behavior:
while we only have year or month, it outputs IntervalYearMonth
❯ select interval '1 year';
+------------------------------------------------+
| IntervalYearMonth("12")                        |
+------------------------------------------------+
| 1 years 0 mons 0 days 0 hours 0 mins 0.00 secs |
+------------------------------------------------+
1 row in set. Query took 0.001 seconds.
❯ select interval '1 year 1 month';
+------------------------------------------------+
| IntervalYearMonth("13")                        |
+------------------------------------------------+
| 1 years 1 mons 0 days 0 hours 0 mins 0.00 secs |
+------------------------------------------------+

while we have (year or month) and any other unit, it outputs IntervalMonthDayNano
+--------------------------------------------------------+
| IntervalMonthDayNano("950737950189618795196236955648") |
+--------------------------------------------------------+
| 0 years 12 mons 1 days 0 hours 0 mins 0.00 secs        |
+--------------------------------------------------------+
1 row in set. Query took 0.001 seconds.
❯ select interval '1 year 1 second';
+--------------------------------------------------------+
| IntervalMonthDayNano("950737950171172051123527404032") |
+--------------------------------------------------------+
| 0 years 12 mons 0 days 0 hours 0 mins 1.00 secs        |
+--------------------------------------------------------+
1 row in set. Query took 0.001 seconds.

Otherwise, it outputs IntervalDaytime
❯ select interval '1 second';
+------------------------------------------------+
| IntervalDayTime("1000")                        |
+------------------------------------------------+
| 0 years 0 mons 0 days 0 hours 0 mins 1.00 secs |
+------------------------------------------------+
1 row in set. Query took 0.001 seconds.
❯ select interval '1 day 1 second';
+------------------------------------------------+
| IntervalDayTime("4294968296")                  |
+------------------------------------------------+
| 0 years 0 mons 1 days 0 hours 0 mins 1.00 secs |
+------------------------------------------------+
1 row in set. Query took 0.001 seconds.

Expected behavior
A clear and concise description of what you expected to happen.
they should be consistent, as our current Timestamp is in nanosecond, i suggest using IntervalMonthDayNano. We could consider add functions to output specific data type(i.e. to_interval_monthdaynano(), to_interval_daytime(), interval_yearmonth() . But i think they might need different arithmetics
Additional context
Add any other context about the problem here.

just found that it's designed in purpose. let me close this and fire another ticket for discussing


Register Function Pointer Tasks on Python Processors

We have some Realm tasks that need to invoke CFFI callbacks on Realm Python processors from C/C++. That requires being able to do register function pointers as tasks on Realm Python processors.

This went into master with commit 1d4d32e in time for the 2020.09 release.


docs parser bumps

Add * stripping for leading strings on new lines, allow */ to  terminate the string, and allow empty descriptions for params and returns.

fix verified on using app

seq for correct prioritisation (longest matching approach is probably better) in other parsers


Add section in the docs for transaction dependencies estimation

We now support tx dependencies estimation, but looks like the docs don't mention it. We should add a section for it.

Yup! Noticed this today when I was answering this question and had to look at some tests to figure out the syntax.


Enum for integer attributes

Hello,
I was a little bit confused with next scenario:
2.3.0 :001 > require 'dry-data'
 => true
2.3.0 :002 > billing_period = Dry::Data['int'].enum(1, 3, 6, 12)
 => #<Dry::Data::Enum:0x007fe1fc85de88 @type=#<Dry::Data::Constrained:0x007fe1fc85ded8 @type=#<Dry::Data::Type constructor=#<Method: Dry::Data::Type.constructor> options={:primitive=>Integer}>, @options={:rule=>#<Dry::Logic::Rule::Value name=Integer predicate=#<Dry::Logic::Predicate id=:inclusion?>>}, @rule=#<Dry::Logic::Rule::Value name=Integer predicate=#<Dry::Logic::Predicate id=:inclusion?>>>, @options={:values=>[1, 3, 6, 12]}, @values=[1, 3, 6, 12]>
2.3.0 :003 > billing_period[3]
 => 12
2.3.0 :004 >

Here you can see that due to special Fixnum processing actual billing period was treated as an array index. That led me to some strange troubles. My spec builds model then persists it (occasionally with wrong value 12 instead of 3). After that when I try to fetch that record from DB the spec fails with an error "nil violates constraints" because it reads 12 and then tries to construct a value by calling PeriodType[12].
It is quite confusing, isn't it? What do you think? :)
P.S. Piotr I want to thank you very much for all that cool stuff. Great work!
Nikita

Hey Nikita, thanks for this gentle bug report, I will fix it in the next version :)


A way to push message early in numaflow instead of waiting until function return

Summary
What change needs making?
Add a code in sdk so user can push the message early in the numaflow map/reduce function instead of at the end of functions
Use Cases
When would you use this?
In flat map situation (1 message generate multiple messages to the downstream vertices),
When we completely generate 1 new message from flat map, we would like to push it to the buffer ASAP because it would take a few minutes until the next message.
Some pseudocode:
import functions from numaflow

...
for f in as_completed(futures_of items):
    functions.Messages.push(Message('success', f.result().as_bytes()))

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

The proposed of the work is,

Introduce a feature flag when it is on, we will applyUDFStream instead if applyUDF to stream the message to the following vertices.
Introduce a new API MapFnStream with gRPC streaming, which needs to be used together with the feature flag.
Python gPRC streaming is slower than unary but it should be fine for flat map use case which takes minutes to generate a message.

With this there are some semantic implications,

[exact once] since this use case is mainly for improvement when a message would take times to generate, that means there is high chance of duplicate message that is not deduped because we have a window for dedup to maintain exact once semantic. That means we may only guarantee at least once semantic in this case. However it should be fine as this may be most for batch-like style use case which the users can just restart the pipeline.
[data in order] with the streaming change, if user want to strict in order data processing, which they just use one pod, the order cannot be maintained. So they might only use it with a batch size of 1.
The watermark propagation should stay intact, as we only do it when a batch is done.


Reduce number of StrictMode warnings

To alleviate some of #6060.
cWRP was easy enough to change. Some instances of cWM weren’t quite as easy due to the legacy context API and having no familiarity with it.
Still, I was able to run this symlinked to my company’s app and see at least a reduction in StrictMode errors with no change in app functionality.
Also added a StrictMode test for each component, skipped if it wasn’t fixed yet.

Thanks for the PR, @jeffchheng  :)
I did some major refactoring over the past week and eliminated all the deprecated componentWill methods in the process, so we won't need this anymore. You can see some of my commits here and here.
My apologies for not guiding you through this work so we could get this merged, but it was a little tricky due to the changes in the context API and I wanted to make sure I understood all of the implications of making this change, so I decided to do it myself. You can expect this to be published in our next beta release sometime this week. 😅

I'm going to cherry-pick the <StrictMode> tests from this PR under a feature flag so we can make sure we don't print any errors in strict mode.

Sorry, I didn't mean to re-open. I'll go ahead and cherry-pick those tests myself.


[AllocineFRBridge] Update CSS class

Website has change one CSS class : updated the bridge to allow parsing.

gj @sysadminstory !


Compile tensorrt inference server source code

Has anyone tried to compile python-tensorrt inference server?
i try to compile but i cannot.
I need some advice. Thanks

Support questions should be asked on the cython-users mailing list rather than reported as bugs here. You might also try asking on the support channel for tensorrt instead.


Maximum Call stack size exceeded when using CollectionFS url() - 1.3.1 syntax

this.helpers({
  image() { return Images.findOne({ _id: imageId }) }
})

It still displays the image but going to the route takes a bit longer than it should while it logs maximum call stack size exceeded several times. It also seems to log it every digest cycle.

If you've just switched from v1.2 to v1.3.1, check in your code if you still have old style searching for image records with underscore. If you do, it will cause Max Call stack size exceeded error.
From v1.3 all searches, writes, etc, should be executed directly upon collection.

@MilosStanic is right, Angular-Meteor 1.2 CollectionFS wrapper breaks the 1.3.
Also, please try to return the .url() value of the image you try to fetch.

I have updated the entire app to the new syntax, but I am still getting this error.

@vacarsu as a personal insight: I'm not using FS.File methods from templates anymore. One of the main reasons was that I figured out that .url() method is not ready immediately upon upload and I wasn't able to fetch images' urls to write them to my other collections. So when I upload, I ping the server to ask if file isStored() and then when it is, I fetch the url from server, and write it in my collection/model. That way I can directly ask the server for images via url, and without calling .url() all the time.

@vacarsu is it an Angular issue or an CollectionFS issue?
Can you share a repo with the problem?

Fixed in 1.3.2


crash in HttpParser.h fenceAndConsumePostPadded

need safe check ?
data[length] = '\r';
data[length + 1] = 'a';

This report is useless


Refactored expected class to remove unexpected and simplify usage

Make the template parameter of expected to be <T> instead of <T, E>: expected is designed to contain either a success value or a fail status, thus no need to make Status a template parameter. I don't see E to be other classes in current code and I don't aforesee any future other E candidates.
Remove unexpected class and allow expected to be constructed directly from Status. So we can do just return Status::some_error: instead of return unexpected(Status:some_error);;
Added 2 macros for easier error status handling(avoids tons of boilerplate code):

// Functions that return `expected<T>`
// without this pr:
auto res = CallSomeFunc();
if (!res.has_value()) {
  LOG(...);
  return res.error();
}
auto res_value = res.value();

// with this pr:
ASSIGN_OR_RETURN(T, res_value, LOG(...), CallSomeFunc());

// Functions that return Status
// without this pr:
Status res = CallSomeFunc();
if (res != Status::success) {
  return res;
}

// with this pr:
RETURN_IF_ERROR(CallSomeFunc());

slightly refactored index.h to extract some duplicate codes into a method
in diskann.cc make TryDiskANNCall non-template and make its argument function have no return value, which is clearer.

This PR contains no feature/performance change.

/lgtm

/kind improvement
/lgtm
/approve

/lgtm


[Snyk] Upgrade @tanstack/react-query from 4.2.3 to 4.3.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @tanstack/react-query from 4.2.3 to 4.3.1.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 6 versions ahead of your current version.
The recommended version was released 22 days ago, on 2022-09-10.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Base: 82.03% // Head: 82.03% // No change to project coverage :thumbsup:

Coverage data is based on head (64fddea) compared to base (9cf508e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #362   +/-   ##
=======================================
  Coverage   82.03%   82.03%
=======================================
  Files          31       31
  Lines         412      412
  Branches       59       59
=======================================
  Hits          338      338
  Misses         68       68
  Partials        6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


just 2 typos

I know they are small but love improving documentation and giving back in any way. Awesome tool, thank you!

Thank you!


Not Booting to DSU

Hello I have an infinix  zero x neo running Android 11 non rooted   when I click on restart  my phone restarts But Boot me Back in you my original Os not the Dsu os

Your device's bootloader is unlocked? if yes, then try to disable boot verification by flashing vbmeta, in most devices you should follow these steps: https://github.com/VegaBobo/DSU-Sideloader/issues/15#issuecomment-1094633182
** i cannot guarantee if those steps will work in your device, i have no clue about your device, neither this brand.
Unlocking bootloader is a requirement, if you aren't unlocked yet, unlock it before trying to boot GSIs.

Thank you


Replace implicit conversions in UiaOperationAbstraction with explicit ones

Types in UiaOperationAbstraction can often be converted to their local and remote variants implicitly. For example, passing a UiaString to a function that takes a wil::shared_bstr will perform the conversion automatically due to the UiaString method operator wil::shared_bstr() const;. These conversions throw if you try to convert a remote abstraction instance into its local variant, or a local abstraction instance into its remote variant. This can result in accidental and often confusing bugs.
It's worth having discussion on whether we want to move towards more explicit conversions to avoid these subtle bugs. This would be a breaking change for UiaOperationAbstraction consumers.

@michaelDCurran @zhayuli

I actually like the implicit conversions but understand the counter arguments. Could the conversions all be placed in one of the headers, if they aren't already, and surrounded  with ifdefs?  I've not looked at the code recently enough to know if this could also be dealt with by putting the conversions in their own namespace which the client could "use" if they're wanted.


Build

certificate-card responsive within the accordion

improvements -> build -> main: certificate card responsive within accordion


update tag/linkback #12

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

update tag/linkback

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[ ] Use the notebook template as a starting point.
[ ] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[ ] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[ ] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

InternalServerError: 500 INTERNAL
Retry, see if happens again

/gcbrun

Step #5: Step #4: RuntimeError: Training failed with:
Step #5: Step #4: code: 9
Step #5: Step #4: message: "Dataset<PHONE_NUMBER>416,<PHONE_NUMBER>350224384) has no Data Item. Please import more Data Items."

Filed bug: 270404719

try with 1.23.0 release
/gcbrun


Update contributing doc and Github URLs

Sync the contributing doc with chef/chef doc (where it makes sense) and update the github URLs to the new orgs.

@chef/client-core Super simple one here

:+1: :shipit:


frontend: at end of onboarding flow: option to register device to unlock the wallet

This is not applicable now.


Support for Gvim + terminator

Hey guys,
I cant find any support for Gvim (vim-gnome) and terminator. Is where any possibility to run test from Gvim in terminator like Mvim on Iterm on OSX?
If yes, can you put it on the readme please.

Right now this plugin only supports OSX terminal and iTerm for GUI vim. We'd welcome a PR for other operating systems and consoles.

+1 for GVim. Can you @gylaz give a hint on why it doesn't work out of Mac?


darling-dmg: 1.0.4+git20180914 -> 1.0.4+git20200427

Motivation for this change
Update darling-dmg to the latest git version, enable LZFSE support to be able to decompress DMGs compressed with LZFSE and add myself as maintainer.
I tried to execute the test built into darling-dmg but that would have added boost as a build-time-dependency, which is huge. Adding such a large dependency for a single test is not worth it in my opinion.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 101637 run on x86_64-linux 1

1 package built:

darling-dmg

Thanks for merging my older PRs.


Fix REST docs login problem

The REST docs of Opencast have always been publicly available, but the recently added new admin interface seems to have broken this. Going to the REST docs overview page and not being logged in causes an empty page to be displayed.
This patch fixes the issue by moving from using jQuery to implementing the whole functionality in pure JavaScript which isn't really any harder with modern JavaScript.
Your pull request should…

[x] have a concise title
[x] close an accompanying issue if one exists
[x] be against the correct branch
[x] include migration scripts and documentation, if appropriate
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits

I'm also taking a look at #4922 right now. I don't hink making docs.js independent from jquery works in that case, as it uses jquery forms and I don't know how to replace those easily. So I'm currently looking at why it can't find the jquery.min.js anymore.

https://github.com/opencast/opencast/pull/4928 fully fixes things, yay!

I would still like to see this merged oven if jQuery reappears somewhere.
Less dependencies are good.


Can  not use  java-client  from  selenide-appium

java-client as "implementation", for use in code need "API" issue started from  1.5.0  version

@onikitchukGain Just released selenide-appium:1.6.1 with Selenide and Appium declares as api dependencies.

@onikitchukGain Just released selenide-appium:1.6.1 with Selenide and Appium declares as api dependencies.


Implement IsPersistent for BLE radio turned back on

Please, can IsPersistent also be implemented to reconnect a device that was connected when the BLE radio was turned off?
When the BLE radio is turned on, any devices that were connected when it was turned off should reconnect

I have done all I can for the persistent connection.  iOS dictates how this works, not me.


refactor: use maps functions instead of custom implementations

This replaces some of our custom code for map-based operations with almost-standard-library versions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.16%. Comparing base (2b165ea) to head (77c08a2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #940      +/-   ##
==========================================
+ Coverage   63.62%   64.16%   +0.54%
==========================================
  Files         146      146
  Lines       11947     9265    -2682
==========================================
- Hits         7601     5945    -1656
+ Misses       3878     2852    -1026
  Partials      468      468

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: consistent/useful naming

[x]  change ANGULAR2_GOOGLE_MAPS_* to GOOGLE_MAPS_*
[x] change GoogleMapsAPIWrapper.getMap to -> GoogleMapsAPIWrapper.getNativeMap

Also change  to  ..:)

@rijine hmm no, I don't want to use the google- prefix. As long it is not possible to overwrite a component selector, I'll stick with my prefix.

Landed as f2084ddfbcec86404bb82209fec5206a267cfb77 and 8efa96d849e43785db0d4b865ea82030f9fe9ac1


Added solid.blockcasting.io

solid.blockcasting.io is a public solid server deployed in Tokyo.
Maintained by Blockcasting Corp.
Added it.

Do I need to submit more information about Blockcasting POD or any others? thanks @kjetilk @RubenVerborgh

Hi @Mitzi-Laszlo , please have a look at this POD.


The program stops running.

After installing posbot, the GD program sometimes stops working.especially when changing fps。
（My English is not very good, so maybe the description is not very clear）
【Video here：https://www.youtube.com/watch?v=fH6fU246lZQ】

This happens when you set your fps to 0
On Fri, 10 Jun 2022 at 04:41, FunokeCN @.***> wrote:

After installing posbot, the GD program sometimes stops working.
（My English is not very good, so maybe the description is not very clear）
【Video here：https://www.youtube.com/watch?v=fH6fU246lZQ】
—
Reply to this email directly, view it on GitHub
https://github.com/decoder0007/PosBor/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMASUCIPOVAVEPXLKNQGPBTVOK2QLANCNFSM5YMINUYQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fixes deprecation warnings

@kittinunf I don't know what is wrong with the CI for android, but I'm merging it.


Allow PyB projects to be installed via URL-based install

Currently PyB allows itself to be installed via pip install <url> by means of a setup file. Unfortunately other projects using PyB cannot take advantage of this without copying the file.
Therefore,

[ ] See if setup.py can be cleaned up and/or simplified.
[ ] Add a task into the core that (re-)generates setup.py file to provide URL-based install to any project.

Good points. Maybe we can integrate this with the scaffolding module (pyb --start-project). Kinda like when starting a scala project you get the option of embedding a SBT launcher, also very much like the gradle wrapper which is created by idea when you set up a gradle project.

Yep, --start-project seems like a reasonable place for this. But we also need to allow regenerating of the file if we have to refresh it. Can --start-project be rerun idempotently without breaking what's already there?

No, it's pretty basic templating. We would have to go through the AST somehow in order to regenerate, but even then it's not easy because project.set_property() does not look like getattr(project, "set_property")() in the AST but it's actually the same thing. So it would be really hard to not re-add things that are already there.
Maybe we can have both: a task that (re)generates the setup.py and --start-project simply calls this when it's done.

Yeah, I'm for the latter approach I think.

depends on #263


NERDTreeSortOrder doesn't seem to be respected after the first element in the list.

If I alter the sort order to something like ['\.cpp$', '\/$'] to have the cpp files listed before the directories, only the cpp files are listed first, and then the other files, along with the directories, are listed in alphabetical order.
http://i.imgur.com/kJ0fLX3.png
At first I thought it was a typo or something similar, but if I re-order the list, so that the directories are first, and the cpp files are second, the directories are correctly placed first, but the cpp files are mixed in with the rest of the files in the directory.
http://i.imgur.com/tuQiUHY.png

Confirmed.

This should be fixed with #856. @tankorsmash , take another look at this, and reopen this issue if necessary.

Thanks @PhilRunninger!


Account button does nothing.

This is the exact same as https://github.com/microsoft/vscode/issues/145445 (but I can't ask there it's closed).
Version: 1.71.2
Commit: 74b1f979648cc44d385a2286793c226e611f59e7
Date: 2022-09-14T21:12:14.256Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.11.0-49-generic
Sandboxed: No

Steps to Reproduce:
1.Click on accounts button in side bar
The click should show a popup for the account settings where i can sync my settings.
Instead, nothing happens.
Some notes:
A. I tried using vscode-insiders, the button works there.
B. Another issue suggests installing gnome-keyring. I checked and it's already installed. Also, if this was the issue I expect it wouldn't work in vscode-insiders, but it does.
Thank you very much in advance.
Best regards.

Started vscode with
code --verbose
clicking on the account button yields:
[1895300:0925/225800.957854:INFO:CONSOLE(603)] "%cTRACE color: #888 [text file model] resolve() - enter file:///home/arthur/dev/btc/champs/test/player-shared-vs-fast.test.ts", source: vscode-file://vscode-app/usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js (603)
[1895300:0925/225800.958097:INFO:CONSOLE(603)] "%cTRACE color: #888 [text file model] resolveFromFile() file:///home/arthur/dev/btc/champs/test/player-shared-vs-fast.test.ts", source: vscode-file://vscode-app/usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js (603)
[main 2022-09-25T20:58:01.169Z] menubarService#updateMenubar 1
[1895300:0925/225801.209451:INFO:CONSOLE(603)] "%cTRACE color: #888 CommandService#executeCommand setContext", source: vscode-file://vscode-app/usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js (603)
[1895300:0925/225801.209903:INFO:CONSOLE(603)] "%cTRACE color: #888 [DEBOUNCE: CodeLensProvide] for file:///home/arthur/dev/btc/champs/test/player-shared-vs-fast.test.ts is 250ms", source: vscode-file://vscode-app/usr/share/code/resources/app/out/vs/workbench/workbench.desktop.main.js (603)
[main 2022-09-25T20:58:01.270Z] [Disk FileSystemProvider]: createResourceLock() - request to acquire resource lock (/home/arthur/.config/Code/User/globalStorage/storage.json)
[main 2022-09-25T20:58:01.270Z] [Disk FileSystemProvider]: createResourceLock() - new resource lock created (/home/arthur/.config/Code/User/globalStorage/storage.json)
[main 2022-09-25T20:58:01.270Z] [Disk FileSystemProvider]: open() - storing lock for handle 137 (/home/arthur/.config/Code/User/globalStorage/storage.json)
[main 2022-09-25T20:58:01.280Z] [Disk FileSystemProvider]: close() - resource lock removed from handle-lock map 137
[main 2022-09-25T20:58:01.280Z] [Disk FileSystemProvider]: close() - disposing lock for handle 137
[main 2022-09-25T20:58:01.280Z] [Disk FileSystemProvider]: createResourceLock() - resource lock dispose() (/home/arthur/.config/Code/User/globalStorage/storage.json)
[main 2022-09-25T20:58:01.280Z] [Disk FileSystemProvider]: createResourceLock() - resource lock removed from resource-lock map (/home/arthur/.config/Code/User/globalStorage/storage.json)
[main 2022-09-25T20:58:01.280Z] [Disk FileSystemProvider]: createResourceLock() - resource lock barrier open() (/home/arthur/.config/Code/User/globalStorage/storage.json)

Doing an ack search for "copilot" in the .config/Code folder:
╭─arthur at aquarelle in ~/.config/Code 22-09-25 - 23:09:16
╰─○ ack copilot
logs/20220925T032624/sharedprocess.log
4:[2022-09-25 03:27:32.886] [sharedprocess] [info] Getting Manifest... github.copilot
5:[2022-09-25 03:27:32.889] [sharedprocess] [info] Installing extension: github.copilot
6:[2022-09-25 03:27:34.062] [sharedprocess] [info] Downloaded extension: github.copilot /home/arthur/.config/Code/CachedExtensionVSIXs/github.copilot-1.46.6822
7:[2022-09-25 03:27:34.115] [sharedprocess] [info] Extracted extension to /home/arthur/.vscode/extensions/.184c6328-2894-4077-a8b1-40db5541f1b5: github.copilot
8:[2022-09-25 03:27:34.124] [sharedprocess] [info] Renamed to /home/arthur/.vscode/extensions/github.copilot-1.46.6822
9:[2022-09-25 03:27:34.125] [sharedprocess] [info] Extracting completed. github.copilot
10:[2022-09-25 03:27:34.126] [sharedprocess] [info] Extension installed successfully: github.copilot

logs/20220925T032624/exthost1/exthost.log
30:[2022-09-25 03:27:34.234] [exthost] [info] ExtensionService#_doActivateExtension GitHub.copilot, startup: false, activationEvent: 'onStartupFinished'
45:[2022-09-25 03:27:40.732] [exthost] [info] ExtensionService#_doActivateExtension GitHub.copilot, startup: false, activationEvent: 'onStartupFinished'

logs/20220925T224833/sharedprocess.log
2:[2022-09-25 22:49:21.081] [sharedprocess] [info] Uninstalling extension<EMAIL_ADDRESS>3:[2022-09-25 22:49:21.169] [sharedprocess] [info] Detected extension removed from another source github.copilot
4:[2022-09-25 22:49:21.359] [sharedprocess] [info] Successfully uninstalled extension<EMAIL_ADDRESS>
logs/20220925T224833/exthost1/exthost.log
15:[2022-09-25 22:48:36.106] [exthost] [info] ExtensionService#_doActivateExtension GitHub.copilot, startup: false, activationEvent: 'onStartupFinished'

logs/20220925T224833/renderer1.log
1:[2022-09-25 22:49:27.161] [renderer1] [error] RequestService#request (browser) - error https://github.gallerycdn.vsassets.io/extensions/github/copilot-nightly/1.46.6863/1663983537473/Microsoft.VisualStudio.Code.Manifest {}
2:[2022-09-25 22:49:29.520] [renderer1] [error] RequestService#request (browser) - error https://benjaminbenais.gallerycdn.vsassets.io/extensions/benjaminbenais/copilot-theme/1.0.0/1649287980501/Microsoft.VisualStudio.Services.Content.Details {}
3:[2022-09-25 22:49:29.520] [renderer1] [error] RequestService#request (browser) - error https://github.gallerycdn.vsassets.io/extensions/github/copilot-labs/0.2.221/1656450740842/Microsoft.VisualStudio.Code.Manifest {}

Same request, with extra context:
╭─arthur at aquarelle in ~/.config/Code/logs/20220925T032624 22-09-25 - 23:16:57
╰─○ ack copilot -A 10 -B 10
sharedprocess.log
1-[2022-09-25 03:26:29.601] [sharedprocess] [info] Starting worker process with pid 1400013 (type: fileWatcher, window: 1).
2-[2022-09-25 03:27:20.886] [sharedprocess] [info] Worker process with pid 1400013 terminated normally (type: fileWatcher, window: 1).
3-[2022-09-25 03:27:21.472] [sharedprocess] [info] Starting worker process with pid 1400318 (type: fileWatcher, window: 1).
4:[2022-09-25 03:27:32.886] [sharedprocess] [info] Getting Manifest... github.copilot
5:[2022-09-25 03:27:32.889] [sharedprocess] [info] Installing extension: github.copilot
6:[2022-09-25 03:27:34.062] [sharedprocess] [info] Downloaded extension: github.copilot /home/arthur/.config/Code/CachedExtensionVSIXs/github.copilot-1.46.6822
7:[2022-09-25 03:27:34.115] [sharedprocess] [info] Extracted extension to /home/arthur/.vscode/extensions/.184c6328-2894-4077-a8b1-40db5541f1b5: github.copilot
8:[2022-09-25 03:27:34.124] [sharedprocess] [info] Renamed to /home/arthur/.vscode/extensions/github.copilot-1.46.6822
9:[2022-09-25 03:27:34.125] [sharedprocess] [info] Extracting completed. github.copilot
10:[2022-09-25 03:27:34.126] [sharedprocess] [info] Extension installed successfully: github.copilot
11-[2022-09-25 03:27:39.516] [sharedprocess] [info] Worker process with pid 1400318 terminated normally (type: fileWatcher, window: 1).
12-[2022-09-25 03:27:40.169] [sharedprocess] [info] Starting worker process with pid 1400619 (type: fileWatcher, window: 1).
13-[2022-09-25 03:31:22.312] [sharedprocess] [info] Getting Manifest... github.github-vscode-theme
14-[2022-09-25 03:31:22.315] [sharedprocess] [info] Installing extension: github.github-vscode-theme
15-[2022-09-25 03:31:23.422] [sharedprocess] [info] Downloaded extension: github.github-vscode-theme /home/arthur/.config/Code/CachedExtensionVSIXs/github.github-vscode-theme-6.3.2
16-[2022-09-25 03:31:23.434] [sharedprocess] [info] Extracted extension to /home/arthur/.vscode/extensions/.eb2d711f-0e7f-4ae6-85ce-9bf159d02822: github.github-vscode-theme
17-[2022-09-25 03:31:23.437] [sharedprocess] [info] Renamed to /home/arthur/.vscode/extensions/github.github-vscode-theme-6.3.2
18-[2022-09-25 03:31:23.438] [sharedprocess] [info] Extracting completed. github.github-vscode-theme
19-[2022-09-25 03:31:23.439] [sharedprocess] [info] Extension installed successfully: github.github-vscode-theme
20-[2022-09-25 03:31:44.640] [sharedprocess] [error] [uncaught exception in sharedProcess]: Cannot read properties of undefined (reading 'then'): TypeError: Cannot read properties of undefined (reading 'then')

exthost1/exthost.log
20-[2022-09-25 03:27:22.023] [exthost] [info] ExtensionService#_doActivateExtension vscode.debug-auto-launch, startup: true, activationEvent: '*'
21-[2022-09-25 03:27:22.025] [exthost] [info] ExtensionService#_doActivateExtension vscode.git-base, startup: true, activationEvent: '*'
22-[2022-09-25 03:27:22.026] [exthost] [info] ExtensionService#_doActivateExtension vscode.ipynb, startup: true, activationEvent: '*'
23-[2022-09-25 03:27:22.028] [exthost] [info] ExtensionService#_doActivateExtension vscode.npm, startup: true, activationEvent: 'workspaceContains:package.json'
24-[2022-09-25 03:27:22.066] [exthost] [info] ExtensionService#_doActivateExtension vscode.github, startup: true, activationEvent: '*'
25-[2022-09-25 03:27:22.085] [exthost] [info] ExtensionService#_doActivateExtension vscode.github-authentication, startup: false, activationEvent: 'onAuthenticationRequest:github'
26-[2022-09-25 03:27:22.094] [exthost] [info] Eager extensions activated
27-[2022-09-25 03:27:22.096] [exthost] [info] ExtensionService#_doActivateExtension vscode.emmet, startup: false, activationEvent: 'onStartupFinished'
28-[2022-09-25 03:27:22.098] [exthost] [info] ExtensionService#_doActivateExtension vscode.merge-conflict, startup: false, activationEvent: 'onStartupFinished'
29-[2022-09-25 03:27:24.305] [exthost] [info] ExtensionService#_doActivateExtension vscode.git, startup: false, activationEvent: 'api', root cause: vscode.github
30:[2022-09-25 03:27:34.234] [exthost] [info] ExtensionService#_doActivateExtension GitHub.copilot, startup: false, activationEvent: 'onStartupFinished'
31-[2022-09-25 03:27:39.456] [exthost] [info] Extension host terminating: received terminate message from renderer
32-[2022-09-25 03:27:39.463] [exthost] [info] Extension host with pid 1400317 exiting with code 0
33-[2022-09-25 03:27:40.588] [exthost] [info] Extension host with pid 1400618 started
34-[2022-09-25 03:27:40.588] [exthost] [info] Skipping acquiring lock for /home/arthur/.config/Code/User/workspaceStorage/f38c6fd23d69e9e65286a9df0cb7c22c.
35-[2022-09-25 03:27:40.632] [exthost] [info] ExtensionService#_doActivateExtension vscode.microsoft-authentication, startup: false, activationEvent: 'onAuthenticationRequest:microsoft'
36-[2022-09-25 03:27:40.641] [exthost] [info] ExtensionService#_doActivateExtension vscode.debug-auto-launch, startup: true, activationEvent: '*'
37-[2022-09-25 03:27:40.642] [exthost] [info] ExtensionService#_doActivateExtension vscode.git-base, startup: true, activationEvent: '*', root cause: vscode.git
38-[2022-09-25 03:27:40.643] [exthost] [info] ExtensionService#_doActivateExtension vscode.ipynb, startup: true, activationEvent: '*'
39-[2022-09-25 03:27:40.645] [exthost] [info] ExtensionService#_doActivateExtension vscode.npm, startup: true, activationEvent: 'workspaceContains:package.json'
40-[2022-09-25 03:27:40.663] [exthost] [info] ExtensionService#_doActivateExtension vscode.git, startup: true, activationEvent: '*'
41-[2022-09-25 03:27:40.674] [exthost] [info] ExtensionService#_doActivateExtension vscode.github, startup: true, activationEvent: '*'
42-[2022-09-25 03:27:40.717] [exthost] [info] Eager extensions activated
43-[2022-09-25 03:27:40.727] [exthost] [info] ExtensionService#_doActivateExtension vscode.emmet, startup: false, activationEvent: 'onStartupFinished'
44-[2022-09-25 03:27:40.731] [exthost] [info] ExtensionService#_doActivateExtension vscode.merge-conflict, startup: false, activationEvent: 'onStartupFinished'
45:[2022-09-25 03:27:40.732] [exthost] [info] ExtensionService#_doActivateExtension GitHub.copilot, startup: false, activationEvent: 'onStartupFinished'
46-[2022-09-25 03:27:40.855] [exthost] [info] ExtensionService#_doActivateExtension vscode.github-authentication, startup: false, activationEvent: 'onAuthenticationRequest:github'
47-[2022-09-25 03:27:52.625] [exthost] [info] ExtensionService#_doActivateExtension vscode.typescript-language-features, startup: false, activationEvent: 'onLanguage:typescript'
48-[2022-09-25 03:30:29.367] [exthost] [info] ExtensionService#_doActivateExtension vscode.configuration-editing, startup: false, activationEvent: 'onLanguage:jsonc'
49-[2022-09-25 03:30:29.368] [exthost] [info] ExtensionService#_doActivateExtension vscode.json-language-features, startup: false, activationEvent: 'onLanguage:jsonc'
50-[2022-09-25 03:33:34.282] [exthost] [info] Extension host terminating: received terminate message from renderer
51-[2022-09-25 03:33:34.304] [exthost] [info] Extension host with pid 1400618 exiting with code 0
╭─arthur at aquarelle in ~/.config/Code/logs/20220925T032624 22-09-25 - 23:17:03
╰─○

Anyone has any idea? This is very handicaping...

Hi @arthurwolf sorry for the delay. Which linux distro are you using?
Glad to hear insiders is fixed for you. We are in the middle of a release, so in just 1 week a new stable will be out. I'm optimistic that this will be fixed then.

╭─arthur at aquarelle in /media/arthur/backup 22-09-28 - 22:58:31
╰─○ uname -a
Linux aquarelle 5.11.0-49-generic #55-Ubuntu SMP Wed Jan 12 17:36:34 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
╭─arthur at aquarelle in /media/arthur/backup 22-09-28 - 23:12:04
╰─○ cat /etc/issue
Ubuntu 21.04 \n \l

This "account button" issue is fixed with -insiders, but the more overall issue, which is that Github Copilot doesn't work, isn't solved by switching to -insiders unfortunately.
Any idea how to get help about Copilot not providing any hints (it did until about a week ago), in either code or code-insiders ? As it's an extension I'm not sure if here is the right place to open an issue.

I don't... however, I think you can get a discussion going here:
https://github.com/community/community/discussions/categories/copilot
And those folks will be able to help!
I'll close this since the account button is fixed in insiders... but if you see it in stable after the next release (in a week) please don't hesitate to reach out!


Raidboss: Titania normal and ex

Added corrections all language.
Added missing translations fr.
Added missing sync fr, ja, cn,  on titania normal.
Added missing sync fr, ja, cn,ko,   on titania ex.
Removed double lines and missingtranslation=true useless.

This is awesome!
But I'm curious that how can you deal with those translation text? Did you make a script to do it automaticlly?

Thank you, the requested changes have been made. No I don't use a script, I'm pretty good at pc, but it's not my job in life, the level here is too high for me ^^. Everything I do, I do it manually. I look at the text file and then I look at the .js file, and I make sure that there are all the lines in replaceSync and replacText. For translations (ja, cn, ko) I only modify or add if I can copy and paste the characters. I do to be very careful not to do unnecessary or wrong things. So much the better if it helps you ^^.


fixing contract id validation in decodeOperation

it looks like Uint8Array are objects in javascript and the '===' is not working, I propose to do the comparison on the base58 encoded version of the contract id (using encodeBase58)

Thanks for the contribution


Temporarily skip test_parquet_read_nano_as_longs_31x on dataproc

Related to https://github.com/NVIDIA/spark-rapids/issues/8074
Temporarily skip test_parquet_read_nano_as_longs_31x on dataproc to unblock the nightly build.

build


Goofy notifications not showing up in notification center

Hi!
First of all, I really like goofy! Super useful :)
Goofy notifications suddenly stopped showing up in notification center in Macbook OS X 10.11. The sound works but the notifications don't show up. My friends are experiencing the same problem. :/

fixed in latest release


Serde is now a non-optional dependency

Removed the enable-serde flag and made Serde a non-optional dependency. Removed the cfg attributes that depended on the feature since now the library is included by default.
Testing
Both unit tests and ./pre-commit pass.
closes private-attribution/ipa#588

I went over the classes that were flagged as part of the coverage check and they were either:
a) Not used anywhere in the code. I removed those classes in the last commits.
b) Tests where not exercising the classes for which I added serialization/deserialization. This issue is orthogonal to this PR and rather tackle that separately.


Add support for extracting stats from raw YCSB output

It may be desirable at some point to have the capability of extracting stats from existing saved YCSB output, rather than only by running YCSB through YCSB-runner. For example, this may be useful if more information needs to be extracted from a previously-run but computationally-intensive experiment.

The Runner.extract_stats classmethod can already extract statistics values from a string, according to the regexps defined in constants.py, however, this is only the initial step. We would need to use Runner.extract_stats to build a StatisticsSet which could then be exported.

Even if it were possible to re-extract stats from captured YCSB output, this would be acceptable. The main problem is having to re-run experiments to get use of the new YCSB-runner features.


Add --amqp_block program argument

This PR adds --amqp_block (disabled by default) program argument, which prevents sg-bridge from dropping amqp messages when the ring buffer is full. This might be useful for debugging (together with --block sg-bridge should deliver everything, that appears on amqp).
Please check if I'm using the pthread condition correctly, it's my first time I'm working with these.

A preliminary scan of the C code looks good.  However, I would be concerned about adding an additional mutex as each additional mutex adds complexity and could add additional possibilities for bugs.  In addition, the requirement to add an additional pthread unlock sequence to all execution paths could cause issues with general performance.  Probably not a whole lot of additional overhead... but every drop in performance is a drop in metrics / second or logs / second.  Maybe the code could be added in a branch and  a build from the branch used for debug.

We decided on a better way to achieve similar functionality, so I'm closing the PR without merging.


Chnaged from the code review

package name changed
switch to mathoid 0.7.0-alpha1
small changes overall

Coverage increased (+3.7%) to 48.607% when pulling 3bc7145c68d87308ce8f9959fcf9cded4ca58df3 on review into b12fe3d04b1d93c8afa58af56f9e5b1c1f7fbe21 on master.

Coverage increased (+22.6%) to 67.492% when pulling 2281be192e1396e9ed682801dc85207ce07936a0 on review into b12fe3d04b1d93c8afa58af56f9e5b1c1f7fbe21 on master.


Issue #172: Slurm role changes and testing framework for Slurm

Signed-off-by: VishnupriyaKrish<EMAIL_ADDRESS>

@VishnupriyaKrish it appears that this PR moves many of the NTP pieces that were removed in #177 and moves them into the Slurm configuration. Any reason for the change from common to Slurm? Does it not make sense to have NTP services regardless of whether Slurm or Kubernetes is installed?

@VishnupriyaKrish it appears that this PR moves many of the NTP pieces that were removed in #177 and moves them into the Slurm configuration. Any reason for the change from common to Slurm? Does it not make sense to have NTP services regardless of whether Slurm or Kubernetes is installed?

@lwilson and @j0hnL
I feel we should keep common, slurm_common, k8s_common separate.
In case of only slurm role is selected we should not install k8s specific files and vice versa.
So we will have following new changes:
common:
ntp
k8s_common:
Add kubernetes repo
Add docker community edition repository
Update sysctl to handle incorrectly routed traffic when iptables is bypassed
Update sysctl
Install docker
Install k8s packages
Versionlock Kubernetes
Start and enable kubernetes – kubelet
@VishnupriyaKrish will modify the code and make the changes once again.

@lwilson and @j0hnL
I feel we should keep common, slurm_common, k8s_common separate.
In case of only slurm role is selected we should not install k8s specific files and vice versa.
So we will have following new changes:
common:
ntp
k8s_common:
Add kubernetes repo
Add docker community edition repository
Update sysctl to handle incorrectly routed traffic when iptables is bypassed
Update sysctl
Install docker
Install k8s packages
Versionlock Kubernetes
Start and enable kubernetes – kubelet
@VishnupriyaKrish will modify the code and make the changes once again.

@VishnupriyaKrish @sujit-jadhav since we are making changes to this PR, I have changed the title to include "WIP:". This will indicate to @j0hnL and I that the PR is a work-in-progress and that it shouldn't be accepted. When you are done making updates, please remove the "WIP:" from the title.

@VishnupriyaKrish @sujit-jadhav since we are making changes to this PR, I have changed the title to include "WIP:". This will indicate to @j0hnL and I that the PR is a work-in-progress and that it shouldn't be accepted. When you are done making updates, please remove the "WIP:" from the title.

@lwilson I have pushed all the required changes

@lwilson I have pushed all the required changes


First step of cleaning up markdown

Implmement first step of formatting markdown to passing linting. I used:
markdownlint --config .github/linters/.markdown-lint.yml \
    --fix docs/Compiler_Hardening_Guides/Compiler-Options-Hardening-Guide-for-C-and-C++.md

I also removed trailing spaces on one line.

@ran-dall - here's a quick first cut at making the markdown pass markdownlint.
I ran this command:
markdownlint --config .github/linters/.markdown-lint.yml \
    --fix docs/Compiler_Hardening_Guides/Compiler-Options-Hardening-Guide-for-C-and-C++.md

Hopefully that leaves you with little to do :-).

Almost al the remaining style complaints were MD049/emphasis-style. We could allow that, or fix those few cases.
There's one case of MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "~~~~"].

I see no objections, and this is just a formal cleanup (without technical changes), so I'm going to merge this.


Apollo Client

I just started on a new app for Shopify and as I'm new to Apollo I was really surprise to see that react-apollo is not really supported. Apollo is asking to use @apollo-client instead.
Is this something you are planning to update?

I have the same issue

I was experimenting with using @apollo/client as well. This works for me:

// _app.js

import { ApolloClient, ApolloProvider, InMemoryCache, createHttpLink } from '@apollo/client';

//...

function MyProvider(props) {
  const app = useAppBridge();

  const client = new ApolloClient({
    cache: new InMemoryCache(),
    link: new createHttpLink({
      credentials: 'include',
      headers: {
        "Content-Type": "application/graphql"
      },
      fetch: authenticatedFetch(app)
    })
  });

  const Component = props.Component;

  return (
    <ApolloProvider client={client}>
      <Component {...props} />
    </ApolloProvider>
  );
}

And using useQuery in a component:
import { gql, useQuery } from '@apollo/client';

const GET_SHOP_NAME = gql`
query {
  shop {
    id
    name
    email
  }
}
`

export default function ShopInfo() {
  const { loading, error, data } = useQuery(GET_SHOP_NAME);

  if (loading) return
Loading...
;
  if (error) return
{error.message}

  console.log(data)
  return (

OK

  );
}

@milancermak

I was experimenting with using @apollo/client as well. This works for me:
// _app.js

import { ApolloClient, ApolloProvider, InMemoryCache, createHttpLink } from '@apollo/client';

//...

function MyProvider(props) {
  const app = useAppBridge();

  const client = new ApolloClient({
    cache: new InMemoryCache(),
    link: createHttpLink({
      credentials: 'include',
      headers: {
        "Content-Type": "application/graphql"
      },
      fetch: authenticatedFetch(app)
    })
  });

  const Component = props.Component;

  return (
    <ApolloProvider client={client}>
      <Component {...props} />
    </ApolloProvider>
  );
}

And using useQuery in a component:
import { gql, useQuery } from '@apollo/client';

const GET_SHOP_NAME = gql`
query {
  shop {
    id
    name
    email
  }
}
`

export default function ShopInfo() {
  const { loading, error, data } = useQuery(GET_SHOP_NAME);

  if (loading) return
Loading...
;
  if (error) return
{error.message}

  console.log(data)
  return (

OK

  );
}

works fine

@milancermak  Hi, I also use new Apollo/Client, but when do a query throw useQuery , the graphql post request occured cros error. I see your code in new ApolloClient ,you ad a authenticatedFetch, Does this line must be included ?

any update of this issue ?? @paulomarg  i'm new to shopify app dev, find the react-apollo has been deprecated. I try to change it to Apollo/Client, but face query request keep loading , no error and no data return .

The code of @milancermak worked fine for me

The code of @milancermak worked fine for me

thanks

The code of @milancermak worked fine for me

thanks

I keep getting this error even while using the code provided by @milancermak
Invariant Violation: Could not find "client" in the context or passed in as an option. Wrap the root component in an <ApolloProvider>, or pass an ApolloClient instance in via options.


Unreal 5.2 is not supported

Currently, in the new source version of Unreal (5.2), GltfRuntime does not compile due to some changes to skeletal meshes in the engine. Most, of the issues, can be fixed by including Engine/SkinnedAssetCommon.h in glTFRuntimeParserSkeletalMeshes.cpp, as most of the objects of skinned assets were moved there. Also AudioExtensions needs to be added to the Build.cs file.
However the function AnimSequence->GetDataModel()->PostDuplicate(false); was also moved in the data model of AnimSequence. This function was used in glTFRuntimeParserSkeletalMeshes.cpp as a hack to call GenerateTransientData(), a function removed in Unreal 5.1. Now I do not know why this function needs to be called, but an alternative solution must be found if this is still needed.
I put all of these suggested changes in PR #54

that .cpp file requires #include "Engine/SkeletalMeshLODSettings.h" I added and created a pull request. I can use it right now.
You can try it yourself
https://github.com/rdeioris/glTFRuntime/pull/67/commits/7414e54510c96d8d09a2d66e3a1ac74c2610df26


feat: port GroupTheory.Archimedean

porting note: I just used classical to get a Decidable instance for membership. I think there is nothing wrong with this, but thought I would mention it just in case.

bors merge


New site

Here comes the new template for your portfolio.
It currently looks something like this:

It looks very good !!


fix(Ruby): gemfile.lock -> Gemfile.lock

What Type of PR is This?

Fixes typo in Gemfile lockfile name

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally (N/A)
[ ] Optional Screen Shoot. (N/A)

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://658c875b1c56a72dc59b0d41--harness-developer.netlify.app

This is in multiple places. I will fix on all and merge in a separate PR.

This is in multiple places. I will fix on all and merge in a separate PR.

Thanks!


AdSize.fluid shows only [Test Ad], but no banner content

Plugin Version
google_mobile_ads: ^2.0.1
Steps to Reproduce
Choose AdSize.fluid as size property of a BannerAd and show it as:
    final screenSize = MediaQuery.of(context).size;

    return SizedBox(
      width: screenSize.width,
      height: 250,
      child: AdWidget(ad: _banner),
    );

(the idea here is to increase banner height in devices with large height to width ratio)
Expected results:
The Ad being show at the available size.
Actual results:
Only the Test Ad text, but no ad (and the ad IS loaded): (the grey area at the bottom is the SizedBox with 250px height)

(flutter run --verbose ommited because GitHub is complaining about  body is too long (maximum is 65536 characters))

Logs

Analyzing frontend...
No issues found! (ran in 3.3s)

[✓] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.19044.1889], locale pt-BR)
    • Flutter version 3.0.5 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (6 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Android
    • Platform android-33, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Android
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.3.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.3.32811.315
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.70.2)
    • VS Code at C:\Users\jckodel\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.46.0

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 12 (API 31) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.19044.1889]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 104.0.5112.102
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 104.0.1293.63

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @JCKodel
Please provide a completed and minimal reproducible code sample so that we may verify this.
Also, there is a fluid_example sample code, you can try using it. The result will look like this:

Demo

Thank you!

Hello, in the fluid_example sample code  you'r using this adUnitId: '/6499/example/APIDemo/Fluid'.
what kind of adUnitId is this  ? i tried to replace it with a nativeAdUnitId but i got this error : Ad unit doesn't match format.

@korchix this needs an bannerAdUnitId to work. Nevertheless, the issue with fluid Adsize persist. While both, test ad and real ad are loaded successfully, only test ad is shown.


How to view and edit .gitignore and README.md at root of repo inside Visual Studio 2019 for Mac

After I clone repository https://github.com/wilsonmar/github-emojis
I can't figure out how to navigate above the solution folder to the root repo folder where the .gitignore and README.md files are located and be changed from.

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

ID: 606d4c70-3daf-8b03-9c31-b690927a7136
Version Independent ID: 24a13aaf-3d14-c810-182c-2c2d93fc299b
Content: Setting up a Git Repository - Visual Studio 2019 for Mac
Content Source: mac/set-up-git-repository.md
Product: visual-studio-mac
Technology: vs-ide-general
GitHub Login: @conceptdev
Microsoft Alias: crdun

You can right-click on the solution and Display Options > Show All Files to view files that aren't in the solution (such as .gitignore or readme files).

You cannot browse up folders from where the solution file is. Right-click and choose Reveal in Finder to view/open those files using the editor of your choice. Note that files beginning with a dot . are often hidden by default in macOS.


[feature] add basic height functions and example

Added some basic height/volume functions and example plate definitions that use lambda functions to call these height/volume functions. I think this is a better way of working than having a function per plate.

actually maybe we should consolidate into height_volume_functions.py 🤔

actually maybe we should consolidate into height_volume_functions.py 🤔

Makes sense, I will push

I consolidated the two files, added depreciation warnings, and fixed the imports where vscode could detect them.

thanks @fderop!
if it's ok with you, i'd like to tweet a link to this pr from the plr twitter so people can see what's changing. what's your @ / how should I give credit?

thanks @fderop!
if it's ok with you, i'd like to tweet a link to this pr from the plr twitter so people can see what's changing. what's your @ / how should I give credit?

Oh wow, I'm flattered, you can use @fvderop


I don't see water in oculus quest

Describe the bug
Hi, I'm trying to use crest in unity for oculus quest. The problem is that by starting the apk in oculus, I don't see water while everything works from unity. Maybe I have to set the scene somehow specific for oculus quest?

Thank you in advanced
Screenshots / video
From unity game player:

From Oculus quest:

Log
error:
Compute shader (UpdateShadow): Property (_ShadowMapTexture) at kernel index (0) is not set
Warning:
Virtual Reality SDK Oculus failed to initialize.
Will attempt to enable Oculus instead.
Assets\Oculus\Avatar\Scripts\OvrAvatar.cs(703,34): warning CS0168: The variable 'e' is declared but never used
Warning! Using more than 1 pixel lights on a mobile device may decrease performance severely. You can change the mobile quality settings in 'Project Settings -> Quality Settings'
Versions
Unity 2019.2.17f1
To Reproduce
I simply imported the example scene from the crest package and I inserted the player controller for oculus quest...
Platform
Oculus Quest - Android
Hardware
Oculus Quest

Thanks for posting this.
I haven't seen that error from the shadow stuff before. Could you disable "create shadow data" on the ocean renderer script and try again?

Thank you for your answer.
I have tried but nothing. In the end, I reset everything and did it all over again. Now it works ... maybe some settings lost along the way...

Ah ok sorry for the trouble


Add information about what proportion of samples in a given categorical metadata field are being shown

Idea from Yoshiki. Would help give context as to which fields are particularly "underrepresented" in a given log-ratio.
e.g. a boxplot for "Timepoint" in the Byrd demo, where there are three possible values (B, F, PF)—if only 50% of PF samples are being shown, that's useful information, since it might let you know that the current log-ratio isn't super helpful

IMO this is pretty important -- have noticed some low frequency taxa that are high-ranked for certain fields where the taxa are only present in a couple of samples of that type. Worth adding on tools that make this clearer soon, since boxplots essentially hide this and I don't think swarm plots are going to be added to vega-lite any time soon.
Perhaps adding an optionally hidden  or something that goes through each unique categorical field for the selected x-axis and computes how many samples in that field are shown. Should show absolute numbers and percentages?
an example where this is important is the pyloric caeca rankings for the mackerel dataset; taking log(top 5%/bottom 5%) does give you a boxplot where the p. caeca samples are relatively high, but then you switch off the boxplot and notice that only 3 p. caeca samples are shown.
I'd be ok just showing this for boxplots.


Newspaper not printing summaries of the articles, its returning blanks

 Issue by pintolx
Tue Apr 21 02:43:05 2020
Originally opened as https://github.com/codelucas/newspaper/issues/800

Hello Everyone,
I am using google news to find news articles about relevant companies that I am interested in. I keep the links from the google search in a list, and try to open each link using Newspaper. However, when i try to find specific information about each article, newspaper returns blanks for summaries, but can print authors and published date. here is my code
try:     import requests     import bs4     import ssl     import pandas as pd     from bs4 import BeautifulSoup     import newspaper     from newspaper import Article     import nltk     from datetime import datetime     from datetime import date except Exception as e:     print('Some module is missing')`
url = "https://news.google.com/search?q=amazon&hl=en-US&gl=US&ceid=US%3Aen"
headers = {
'Accept-Encoding': 'gzip, deflate, sdch',
'Accept-Language': 'en-US, en;q=0.8',
'User-Agent': '',
'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8',
'Referer': 'http://www.wikipedia.org/',
'Connection': 'keep-alive',
}
r = requests.get(url=url, headers=headers)
soup = BeautifulSoup(r.text, 'html.parser')
imgV = []
titleV = []
linkV = []
timeV = []
summaryV = []
for x in soup.findAll("div", {'jscontroller': 'jaOnYb'}):
for y in x.findAll("div", class_="NiLAwe y6IFtc R7GTQ keNKEd j7vNaf nID9nc"):
    summary = y.text
   # print(summary)

    summaryV.append(summaryV)

    link = y.find('a')['href']
    base_url = "https://news.google.com" + link[1::]
    linkV.append(base_url)

    title = y.find('h3')
    titleV.append(title.text)
    #print(title.text)

    for z in y.findAll('time'):
        timeV.append(z.text)
        # print(z.teext)
        break

#Accessing the results of the urls - print title, summary and publish_date
import newspaper
from multiprocessing.dummy import Pool as ThreadPool
def getTxt(url):
article = Article(url)
article.download()
try:
article.parse()
j= article.summary
p = article.publish_date
print(j)
print(p)
return p
except:
return ""
pool = ThreadPool(10)
open the urls in their own threads
and return the results
results = pool.map(getTxt, linkV)
close the pool and wait for the work to finish
pool.close()
pool.join()
`

summary is only filled with .nlp() call.
it is not the summary in the article, it is built through NLP by analyisig the parsed text


[REQ]  Filtering of.posts

Hi,
I would like to humbly request for filtering capabilities in Quacker using keywords and wildcards such as
"Crypto*" to filter posts such as Cryptocurrency, Cryptomania etc.
"World Cup" to filter posts that contains this string.
Purpose.of this is to reduce "noise" as not all tweets are relevant and can clog up the timeline.
Thank you!

Hi @TheHCJ ,
Thank you for the response. My apologies as I think I was concise in the ticket. My intent is to have a filter such that those articles are filtered out from the Feed and not to search for them.
As mentioned the intent is to filter those posts so that they won't not appear inside the Feed in the first place.
Thank you


how can i get the before node name?

when i use the below command,i want get the node name before clustering ,how i should program to get it.
result = mc.run_mcl(matrix)
clusters = mc.get_clusters(result)

Please specify your inputs and the output you desire, because i'm not sure what do you mean exactly.

I think I got the same issue ;
I have previously built a network with networkx, from a pandas adjacency table  :
Matrice = pd.read_csv('adj_matrix.csv', sep=',', index_col=0, header=0)
network = nx.from_pandas_adjacency(df=Matrice, create_using=nx.Graph())

My labels are formatted as follow :  'OTUxx' , where xx is a number from 1 to several digits.
      | OTU85 | OTU58 | OTU95 | ...
______________________________________
OTU85 |
OTU58 |          the adjacency table
OTU95 |
...

When I apply the MCL algorithm and draw the clustered graph (with random positions) with nodes labels :
matrix = nx.to_scipy_sparse_matrix(network)
result = mc.run_mcl(matrix)
clusters = mc.get_clusters(result)
positions_random = {i:(random.random() * 2 - 1, random.random() * 2 - 1) for i in range(network.number_of_nodes())}
mc.draw_graph(matrix, clusters, pos=positions_random, node_size=25, with_labels=True, edge_color="silver")

The labels aren't the same anymore : nodes are only labelled by numbers, starting at 0. I don't know if the modification happens in nx.to_scipy_parse(), the mc.run_mc(), or the mc.get_clusters() commands.
How can I make a correspondance between my initial labels and the final labels ?

I see what the problems is thank you @Mataivic .
I'll look into it in the coming days. This issue will stay open meanwhile.
Feel free to submit a trick to avoid the problem if you find on, or a PR if you can add it as an option.

The scipy matrices that are used by the library don't support row and column names,  so that information doesn't get used.
The order of row and columns is unchanged, so the indexes of all nodes is consistent.
I think that the issue can be solved by allowing a list of node labels to be passed to mc.draw_graph.

Hello,
Is that issue solved? I also need node labels or names instead of indices in the results. I cannot draw graph due to my network is really big (~17000 nodes).


Automated cherry pick of #419: Imported golang golint. #417: fix BestEffort Job for overused

Cherry pick of #419 #417 on release-0.2.
#419: Imported golang golint.
#417: fix BestEffort Job for overused

Travis tests have failed
Hey @k82cn,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build
View build log

     make verify

go build -o _output/bin/deepcopy-gen ./cmd/deepcopy-gen/
_output/bin/deepcopy-gen -i ./pkg/apis/scheduling/v1alpha1/ -O zz_generated.deepcopy
hack/verify-gofmt.sh
diff -u ./test/e2e.go.orig ./test/e2e.go
--- ./test/e2e.go.orig	2018-10-15 01:49:34.275942896 +0000
+++ ./test/e2e.go	2018-10-15 01:49:34.275942896 +0000
@@ -221,7 +221,7 @@
 		err = waitTasksNotReady(context, job.Name, nn)
 		Expect(err).NotTo(HaveOccurred())
 	})
-
+
 	It("Schedule BestEffort Job", func() {
 		context := initTestContext()
 		defer cleanupTestContext(context)
make: *** [verify] Error 1

TravisBuddy Request Identifier: 6947ce70-d01e-11e8-8e29-6bd38fa4cac4


fix newline discrepancy

Fix the newline discrepancy between Windows '\r\n' and Linux '\n'.
https://docs.python.org/3/glossary.html#term-universal-newlines

Issue fixed with #25


Werf helm template consumes "Chart.yaml file is missing" error

Calling helm template in an empty directory through werf results in empty string and calling it directly returns an error (as seen in image below).

It happens due to the fact that werf uses chart_extender to modify original chart metadata, and AutosetChartMetadata function doesn't check whether incoming metadata is nil or not.
This PR emulates behavior of the original helm template command.

Thanks!

Looks like it broke all other non-helm commands (werf converge, ...):
  Release "***-test-converge-24-912f565e-test" does not exist. Installing it now.
  Running time 1.57 seconds
  Error: helm upgrade have failed: Chart.yaml file is missing

I guess it was like that because default chart in .helm does not need Chart.yaml to be present for all commands except werf helm ..., since commands like werf converge dynamically generate Chart.yaml if it's missing. I'll revert this change, werf converge must not break.

Okay, got you, will try to come up with a better solution


how to set .env file in windows 10

Hi Dhravya Shah,
please tell me how to set .env file in windows 10,
waiting for your response
finest regards

Hi @elmirarokni you need to create a file with the name .env in the root of the project and then put the environment variables in there


Demonstrate Apollo-Server context usage with-apollo-xxx examples

Feature request
Current with-apollo- examples do not demonstrate how to use Apollo-Server with context resolver. It doesn't work out of the box and it's not at all straightforward for Apollo newbies (like myself) how to enable it.
Note: this is a request for help/consulting for Apollo experts.
1. createIsomorphLink
createIsomorphLink implementations recommended by Apollo Team imply the use of
SchemaLink on server and HttpLink on client. SchemaLink is said to be more performant
because it avoids HTTP layer alltogether:
function createIsomorphLink(ctx) {
  if (typeof window == 'undefined') {
    // !!! Server side
    let {SchemaLink} = require('apollo-link-schema')
    let {schema, context} = require('./schema')
    return new SchemaLink({schema, context : ctx})
  } else {
    // !!! Browser side
    let {HttpLink} = require('apollo-link-http')
    return new HttpLink({
      uri: '/api/graphql',
      credentials: 'same-origin',
    })
  }
}

2. SchemaLink
SchemaLink does not initiate an HTTP request. The side effect is that Apollo-Server context function is never called. This function is crucial as it usually fetches or polyfills all common resolver's data including current user/visitor, etc. Current with-apollo- examples demonstrate how to read user data in local (leaf) resolvers which is not production-like.
Luckily, SchemaLink accepts a context argument where we can provide the same context resolver as to the new ApolloServer({context: ...}).
3. Two calls
Now the tricky part is that createIsomorphLink is called twice. Once with {req, res} context data (coming from getInitialProps) and another – with no context (coming from these:
 const WithApollo = ({ apolloClient, apolloState, ...pageProps }) => {
    const client = apolloClient || initApolloClient(undefined, apolloState) // !!!

lines).
context function of Apollo-Server must not be called with empty context as it, in most cases, reads ctx.req.
The second invocation of Apollo-Client will reuse cache so resolvers, including context won't be called. It seems to me that an update like:
function createIsomorphLink(ctx) {
  ...
-  let {schema} = require('./schema')
-  return new SchemaLink({schema, context: ctx)
+  let {schema, context} = require('./schema')
+  return new SchemaLink({schema, context : isEmpty(ctx) ? context(ctx) : ctx})
}

should be enough to enable Server's context usage. Given that apollo/schema.js exports standalone context function of course.
I can make a PR but, before that, it would be great to get a feedback from someone more experienced with Apollo and NextJS.
Are my reasoning correct? Is my proposal valid?
Another option it to always use HttpLink like it's done in https://github.com/adamsoffer/next-apollo and https://github.com/lfades/next-with-apollo For some reason the authors of both packages decided to not rely on SchemaLink...

cc @adamsoffer @lfades I believe you guys have something to say about it.

I chose to review and approve PRs to update the examples, I've been disconnected from Apollo for a while now, and because it has been changing so much, the community is the one deciding the state of the examples.
If the approach you're mentioning is better and the proof is there, you're welcome to create a PR with the changes.

I was just finding this same issue with the apollo Auth example.
Context will likely want more than just the standard ctx - for example, id like to check JWT authentication token from the cookie in the context function and apply user ID to the context or throw an auth error, so i would recommend always using a context function rather than only when empty, the cache sounds correct though.
return new SchemaLink({schema, context : getContext(ctx)})

@glenarama your code won't work by the reasons I described above.

I should have noted that the getContext returns an empty object if no req is passed, this works fine for me.
Context isn't needed at all on the second pass - certainly in my case your second point is true: You can't rely on fake (empty) ctx in context resolver.
Im passing null on the second pass and it works great.
Another way (clearer) of writing what i had above is:
return new SchemaLink({schema, context : isEmpty(ctx) ? null : getContext(ctx)})

In that case we're on the same page!
null vs {}  as well as the way to check for it are implementation details IMO.

👍
On Thu, Feb 6, 2020, 11:45 AM Ivan Kleshnin<EMAIL_ADDRESS>wrote:

In that case we're on the same page!
null vs {} as well as the way to check for it are implementation details
IMO.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeit/next.js/issues/10413?email_source=notifications&email_token=ABTE7XA6PTRUPA3NH6NWVW3RBPZ5LA5CNFSM4KPTVDC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK65THY#issuecomment-582867359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTE7XE3LJQ3XRZE4YB7RCTRBPZ5LANCNFSM4KPTVDCQ
.

@ivan-kleshnin I fixed your issue (3. Two calls) in the following PR https://github.com/zeit/next.js/pull/10451
Since the example currently doesn't implement a server schema I never added SchemaLink recommendation. Just in case you want to implement this yourself please make sure that the schema won't be shipped to the client.
Hope this helps...

@HaNdTriX yes, you did a great job with that PR! 👍

Hi, just to give feedback we use SchemaLink in Vulcan without much trouble as far as I know. We define different clients for SSR and client-side rendering, literally in different files.
Having worked with isomorphic code and SSR for a while, we now tend to consider calls like "ssr: Boolean(ctx)" to be anti-pattern, except if you really can't avoid it or it's done in a "clean" and limited manner like in Next pages.
This is because server-side code tend to diverge slowly from client-side code, and it becomes unmanageable in an app that is multiple years old (unclear bundling process, require() everywhere). Even if it look alike in the beginning you quickly hit issues like needing different link depending on the environment. The first time you wonder "am I server-side?" in your code is probably the good time to split your file in 2 for client and server.
An important step is to unify context between REST calls (here the API routes), graphQL calls server side, and graphQL calls client-side. For some reason using a context in queries is not that common in traditional Node app. As far as I understand that's the point of this issue.

Btw I do not agree with the "good-first-issue" tag, this is trickier than it seems and raises questions about state-of-the-art patterns like isomorphic code. There is no clear consensus about how to manage such issues.


Change "Check wallet logs" text in wallet settings to "View Log"

[ ] Change "Check wallet logs" text in wallet settings to "View Log"
[ ] Change "Check Statistics" text in wallet settings to "View Stats" or "View Info"
[ ] In wallet settings page, General section has inconsistency on row height

I am working on this


Model weights not saved if a custom layer class contains a list of layers named self._layers

Hi everyone,
I would like to point out a problem that I found while developing a custom model: layer's weights are not saved if a Model subclass initializes  a custom layer that contains sublayers in a class-level list variable called self._layers. The code below proves the issue and should illustrates it better.
import os
from pathlib import Path

import keras
import numpy as np

@keras.saving.register_keras_serializable(package="KerasTest", name="CustomLayer")
class CustomLayer(keras.Layer):

    def __init__(self, bugged: bool = False, name="test_layer", **kwargs):
        super(CustomLayer, self).__init__(name=name, **kwargs)
        self._bugged = bugged
        if self._bugged:
            self._layers = [
                keras.layers.Dense(64, activation='relu'),
                keras.layers.Dense(32, activation='relu'),
                keras.layers.Dense(10, activation='softmax')
            ]
        else:
            self._custom_layers = [
                keras.layers.Dense(64, activation='relu'),
                keras.layers.Dense(32, activation='relu'),
                keras.layers.Dense(10, activation='softmax')
            ]

    def call(self, inputs):
        x = inputs
        layer_list = self._layers if self._bugged else self._custom_layers
        for layer in layer_list:
            x = layer(x)
        return x

    def get_config(self):
        base_config = super().get_config()
        config = {
            "bugged": self._bugged
        }
        return {**base_config, **config}

@keras.saving.register_keras_serializable(package="KerasTest", name="TestModel")
class TestModel(keras.Model):

    def __init__(self, bugged: bool = False, name="test_model", **kwargs):
        super(TestModel, self).__init__(name=name, **kwargs)
        self._bugged = bugged
        self._custom_layer = CustomLayer(bugged=bugged)

    def call(self, inputs):
        return self._custom_layer(inputs)

    def get_config(self):
        base_config = super().get_config()
        config = {
            "bugged": self._bugged
        }
        return {**base_config, **config}

def test_model(bugged: bool = False):
    output_path = Path("./output")
    model_name_prefix = "bugged" if bugged else "fixed"

    # Dataset generation
    num_samples = 1000
    input_data = keras.random.uniform(shape=(num_samples, 32))
    labels = keras.random.randint(minval=0, maxval=10, shape=(num_samples,))
    labels = keras.utils.to_categorical(labels, num_classes=10)

    # Test bugged model
    model = TestModel(bugged=bugged)
    model.compile(optimizer='adam', loss='categorical_crossentropy', metrics=['accuracy'])
    model.fit(input_data, labels, epochs=5, batch_size=32, validation_split=0.2)
    trained_weights = model.get_weights()

    # Save and load model
    output_path.mkdir(parents=True, exist_ok=True)
    model.save(output_path / f"{model_name_prefix}_model.keras")
    loaded_model = keras.saving.load_model(output_path / f"{model_name_prefix}_model.keras")
    loaded_weights = loaded_model.get_weights()

    print(f"------------ {model_name_prefix.capitalize()} - Compare trained weights to loaded weights -------------")
    for i, (uw, tw) in enumerate(zip(trained_weights, loaded_weights)):
        comparison = np.array_equal(uw, tw)
        print(f"Layer {i} -> Weights match: {comparison}")

if __name__ == '__main__':
    os.environ["KERAS_BACKEND"] = "tensorflow"
    test_model(bugged=True)
    test_model(bugged=False)

The output is:
------------ Bugged - Compare trained weights to loaded weights -------------
Layer 0 -> Weights match: False
Layer 1 -> Weights match: False
Layer 2 -> Weights match: False
Layer 3 -> Weights match: False
Layer 4 -> Weights match: False
Layer 5 -> Weights match: False
------------ Fixed - Compare trained weights to loaded weights -------------
Layer 0 -> Weights match: True
Layer 1 -> Weights match: True
Layer 2 -> Weights match: True
Layer 3 -> Weights match: True
Layer 4 -> Weights match: True
Layer 5 -> Weights match: True

I think that the same problem has been solved for the Model class by declaring a setter method (line 170). Perhaps it is possible to use the same approach for the Layer class.

_layers is an internal property and you should not override it. Just use any other name.

Yes now I know, but since it is not documented anywhere and it is not an uncommon name to use, I was suggesting to use the same approach used for the Model class and print an error when trying to set the property from the child class. Because it was not easy to find the error since you only know that something is wrong after the training and you have no errors or warnings.


format: Chinese translation format improve

Most of them is fixing markdown foramt.

@microsoft-github-policy-service agree


Packages by feature

@android10 I read the article which you have written for this example and I read that you prefer and you suggest to use package by feature. We decided to follow this architecture which you described (with a few changes), but as a DevTeam we want to follow this kind of logic (for ownership) that's why  I want to ask you how it can be this kind of approach for package by feature.
Let's say I am in Domain layer and I have the following features authentication, search. I will have the followings:
com.test.domain.authentication
├─ exception
├─ executor
├─ interactor
├─ repository

com.test.domain.search
├─ exception
├─ executor
├─ interactor
├─ repository

Right? The same logic for each layer (data, presentation) in order to be packaged by feature.

Sounds reasonable, and there may need to be more or less packages added per feature in the future so this isn't a hard set rule. The reason those four packages are likely implemented right inside the domain package for this example project was for simplicity. KISS and scale up when you need to.
You could also do the same thing in reverse as well having each feature implemented once inside each of those four packages (exception, executor, interactor, repository). Either way you're going to end up with 4*(# of features) packages.

@caseykulm yes I agree it's not a hard rule, I want to understand the main concept that's why I gave this example.
Yeap you are right, maybe this is cleaner than my approach because the main core of packages remains the same.

I will wait a comment by @android10.

@spirosoik package by feature as you may know will help you out to no overlap with other teams and touch the same code preventing big conflicted merges.
Regarding what you pointed out, you are right by grouping functionalities in packages:
com.test.domain.authentication
com.test.domain.search

The rest of the packages will depend on the granularity you wanna use:
com.test.domain.authentication
├─ exception
├─ executor (this might not be necessary for each feature)
├─ interactor (you can avoid this package and put your use cases right inside the feature)
├─ repository (you can avoid this package and put your repository right inside the feature)

com.test.domain.search
├─ exception
├─ executor (this might not be necessary for each feature)
├─ interactor (you can avoid this package and put your use cases right inside the feature)
├─ repository (you can avoid this package and put your repository right inside the feature)

The reason why I used such high granular approach was more for learning purpose. I could get rid of them since each class name defines its purpose and it would not be hard to read and understand.

@android10 :+1:  excellent thanks for the great info and for the great approach for all stuff. We have already started using the package by feature as you mentioned.

@spirosoik good to hear that! Looking forward to your thoughts and things you find in your way :)


Add bash script for k8s flex driver

This introduces a 'flexrex' script that can be installed and used as a Kubernetes
Flex Volume driver. See: https://github.com/kubernetes/kubernetes/tree/master/examples/volumes/flexvolume
This script makes calls to rexray for attach/detach, but does not use rexray for
mount/umount because the behavior that k8s needs is different from what
is built into the Docker integration driver.
Examples:
rexray volume create test --size 5
flexrex attach '{"volumeID":"test"}'
flexrex mount '/mnt/test' '/dev/sdb' '{"kubernetes.io/fsType":"ext4"}'
flexrex unmount /mnt/test
flexrex detach '/dev/sdb'

I made sure that both xfs and ext4 fs types work. The lvm example given for the flex driver does not work on unpartitioned block devices.
Testing has not been done with k8s proper at this point, only with the flexrex script directly, as described in the examples above.  cc/ @cduchesne
This script only works with rexray starting with v0.6.1, as it requires the volume ls --path functionality`.  Not entirely sure where the script should live at this point, so I just proposed a place. Purpose of PR is to get to @cduchesne for testing.

Current coverage is 8.48% (diff: 100%)

Merging #641 into master will not change coverage

@@            master      #641   diff @@
========================================
  Files           16        16
  Lines         2087      2087
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           177       177
  Misses        1906      1906
  Partials         4         4

Powered by Codecov. Last update 4e778fa...84cbc60


Can't solve Error: "CSRF Token not found. It's possible another plugin is altering requests sent by the Cloudflare plugin."

I've disabled all my plugins and still can't get this error, CSRF Token not found. It's possible another plugin is altering requests sent by the Cloudflare plugin, to resolve.
On version 4.9.4.
Here are what I have installed. Disabling each of these hasn't helped and the error persists:
Advanced Custom Fields
Tracking Code Manager
Bookify
Cloudflare
Faq Responsive
Intergeo - Google Maps Plugin - Lite
MailChimp for WordPress
Max Mega Menu
Ninja Forms
Tracking Code Manager

I've got a similar problem with same Error Message. [ CSRF Token not found. It's possible another plugin is altering requests sent by the Cloudflare plugin .] But in this kind of stuff, like implementing Cloudflare, I'm only an Advanced Newbie, so not really sure where to go on this from here. The six websites I went with SSL via Cloudflare display okay, but mail server on ONE site is not working.
Cloudflare forum is not much help, so far.
I do know pretty well how to get around most problems with my Wordpress sites, but I'm not even sure what questions to ask here.
Thank You Very Much,
David Scott Lynn

I have this error, too.  I get it when I try to apply any of the plugin settings (optimize Cloudflare for wordpress; purge cache; automatic cache management...)  I have a number of plugins, but deactivating them all (except made no difference.  Still the error persisted.
The plugin is version 3.3.2 and the WP version is 4.9.6.

Same error here.
plugins:
Advanced Custom Fields PRO
Akismet Anti-Spam
Coming Soon by Supsystic
Contact Form 7
Easy Google Fonts
Fusion Builder
Fusion Core
LayerSlider WP
Login LockDown
Slider Revolution
UpdraftPlus - Backup/Restore
WP version 4.9.8

CMS Commander has a problem too

The issue its about that if another plugin do file_get_contents('php://input');
Seems like "php://input" it's only readable once


Successful deployment reported despite error

This has been causing me to scratch my head for some time, and while I assume more a core Azure issue, I'm unsure where to report it.
We are maintaining a .NET 6.0 web app, which has and can be successfully published from Visual Studio
Attempt to deploy the project from CI/CD appear successful, they show in the deployment centre and when you FTP onto the box, the deployment and logs appear under /site/deployments, however wwwroot remains untouched, with last modified remaining as the last date deployed through visual studio
There is not trace of error in the github action, however here are the full deployment logs (with some chars replaced with x's):
2023-12-08T16:24:57.8032242Z,Updating submodules.,xxxxxxx-524d-40d9-8447-4f1b8b0b55d,0
2023-12-08T16:24:57.8955282Z,Preparing deployment for commit id 'xxxxxx'.,xxxxxx-6c7c-4897-be85-,0
2023-12-08T16:24:58.3336180Z,Running build. Project type: OneDeploy,xxxxxxx-12fa-40b3-a277-,0
2023-12-08T16:24:58.3954941Z,Copying the manifest,xxxxxx-b52e-455b-b691-,0
2023-12-08T16:24:58.4579820Z,Clean deploying to C:\home\site\wwwroot,xxxxxxx-4622-44bc-899c-,0
2023-12-08T16:24:58.4899385Z,Generating deployment script.,xxxxxxxx-39f2-479d-bb40-,0
	2023-12-08T16:24:58.5531074Z,Using the following command to generate deployment script: 'azure site deploymentscript -y --no-dot-deployment -r "C:\local\Temp\zipdeploy\extracted" -o "C:\home\site\deployments\tools" --basic --sitePath "C:\local\Temp\zipdeploy\extracted"'.,,0
	2023-12-08T16:25:00.1327231Z,C:\PROGRA~2\SITEEX~1\NODEAP~1\296456~1.8\nodejs\build\src\Loader.js:153,,1
	2023-12-08T16:25:00.1509723Z,                ...StatusLogger.DEFAULT_STATUS,,,1
	2023-12-08T16:25:00.1945599Z,                ^^^,,1
	2023-12-08T16:25:00.2279123Z,,,1
	2023-12-08T16:25:00.2413032Z,SyntaxError: Unexpected token ...,,1
	2023-12-08T16:25:00.2725500Z,    at createScript (vm.js:56:10),,1
	2023-12-08T16:25:00.2881783Z,    at Object.runInThisContext (vm.js:97:10),,1
	2023-12-08T16:25:00.3040370Z,    at Module._compile (module.js:542:28),,1
	2023-12-08T16:25:00.3194280Z,    at Object.Module._extensions..js (module.js:579:10),,1
	2023-12-08T16:25:00.3672026Z,    at Module.load (module.js:487:32),,1
	2023-12-08T16:25:00.3975421Z,    at tryModuleLoad (module.js:446:12),,1
	2023-12-08T16:25:00.4600428Z,    at Function.Module._load (module.js:438:3),,1
	2023-12-08T16:25:00.4787040Z,    at Module.require (module.js:497:17),,1
	2023-12-08T16:25:00.5072257Z,    at module.js:669:12,,1
	2023-12-08T16:25:00.5381777Z,    at Array.forEach (native),,1
	2023-12-08T16:25:00.5850253Z,C:\PROGRA~2\SITEEX~1\NODEAP~1\296456~1.8\nodejs\build\src\Loader.js:153\r\n                ...StatusLogger.DEFAULT_STATUS,\r\n                ^^^\r\n\r\nSyntaxError: Unexpected token ...\r\n    at createScript (vm.js:56:10)\r\n    at Object.runInThisContext (vm.js:97:10)\r\n    at Module._compile (module.js:542:28)\r\n    at Object.Module._extensions..js (module.js:579:10)\r\n    at Module.load (module.js:487:32)\r\n    at tryModuleLoad (module.js:446:12)\r\n    at Function.Module._load (module.js:438:3)\r\n    at Module.require (module.js:497:17)\r\n    at module.js:669:12\r\n    at Array.forEach (native)\r\nC:\Program Files (x86)\SiteExtensions\Kudu\100.51108.6494\bin\Scripts\starter.cmd "C:\Program Files (x86)\SiteExtensions\Kudu\100.51108.6494\bin\node_modules\.bin\kuduscript.cmd" -y --no-dot-deployment -r "C:\local\Temp\zipdeploy\extracted" -o "C:\home\site\deployments\tools" --basic --sitePath "C:\local\Temp\zipdeploy\extracted",,2
2023-12-08T16:25:00.6475279Z,Build completed succesfully.,xxxxxxx-cfb4-4728-8ba4-786754602c7f,0
2023-12-08T16:25:00.6943941Z,Running post deployment command(s)...,xxxxxxx-2b5f-4ae0-bc05-14e31a5d1231,0
2023-12-08T16:25:00.7881477Z,Requesting site restart,xxxxxxx-f2e1-475b-89e6-4c671a573b17,0
2023-12-08T16:25:00.8667504Z,Requesting site restart. Attempt #xxxxxxx-281a-4bc1-988c-0413894d6337,0
2023-12-08T16:25:01.4130930Z,Successfully requested a restart. Attempt #1,xxxxxx-10ba-4d1e-b0ec-d6983e174e91,0
2023-12-08T16:25:01.4912170Z,Deployment successful.,xxxxxx-7af7-4699-8c54-0239002cd5dc,0

Presumably the stack trace is related, but as you can see it reports success when it has in fact failed

The problem was resolved, so I can confirm it was related to the stack call (unsurprisingly) updating a node via the WEBSITE_NODE_DEFAULT_VERSION application setting in azure and remembering to save it allowed me to deploy and see changes.
The issue remains open though, as:

The stack trace is caught and the deployment assumed successful
The stack trace was not bubbled up to the Github Action; I had to FTP onto the server and view the log on there

Still facing this


fix: tailwind base styles are duplicated

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
On the docs site, all tailwind base styles were included twice in the bundled .css files. First are added by the tailwind astro integration, second by @tailwind base line in global.css.
This PR adds a config to the astro integration to fix that.

Screenshots

💯

Why is this not default lol???

There's an explanation here, although I do agree this feels like kind of a footgun.
https://docs.astro.build/de/guides/integrations-guide/tailwind/#configapplybasestyles
Thanks for noticing and fixing this :)

There's an explanation here, although I do agree this feels like kind of a footgun.
https://docs.astro.build/de/guides/integrations-guide/tailwind/#configapplybasestyles
Thanks for noticing and fixing this :)

Is it cause we import our css files in the layout?


Add capability to import esm modules in test

Following this discussion on stakoverflow: https://stackoverflow.com/questions/68253518/testcafe-seems-to-not-be-compatible-with-last-version-of-logsymbol/
It seems testcafe is not compatible with pure esm modules.
If we want to enhance our framework to use others pure esm modules, we cannot.
It will be great if testcafe will be a pure esm itself in order to follow future evolve of node and ES.

It's a duplicate of #6853. Let's continue the discussion there.


fix #2: Seperation of configuration and recording commands

The code has been refactored to accommodate the configuration and recording steps separately.
Hi @chuber1986, Could you please build and test this to see if it works ?! I couldn't  test the changes

Hi @azinke ,
I'm currently still on vacation. I will test your fix as soon as I'm back in my office next week.


bad (main) service for codeserver addon when multiple ingress hosts

Details
Helm chart name and version:
3.1.0

What steps did you take and what happened:
Used addon with multiple ingress entries
addons:
      codeserver:
        enabled: true
        volumeMounts:
        - name: config
          mountPath: /config
        ingress:
          enabled: true
          hosts:
          - host: ha-editor.pub.${CLUSTER_DOMAIN}
            paths:
            - path: /
              pathType: Prefix
          - host: ha-editor.home.${CLUSTER_DOMAIN}
            paths:
            - path: /
              pathType: Prefix
          tls:
          - hosts:
            - ha-editor.pub.${CLUSTER_DOMAIN}
            - ha-editor.home.${CLUSTER_DOMAIN}

Relevant Helm values:

What did you expect to happen:
Both ingress entries pointing the vscode
Anything else you would like to add:

The problem comes from
{{- $_ := set (index (index $ingressValues.hosts 0).paths 0) "service" (dict "name" $svcName "port" .Values.addons.codeserver.service.ports.codeserver.port) -}}
It should be a range statement to not only process the first entry.
Additional Information:

Easy to fix - can do a PR - capturing for now as issue to not forget it.

Thanks for raising the issue! Should be fixed with 3.1.1


Add TOC for README

Add TOC for README.

Added with https://github.com/iamazeem/fluent-plugin-protobuf-http/commit/ed602adf125ee567256d70f426f16fefac106cbf.


clean up epmgp.py

it is not easy to read. Also we might want to move it out of the util folder

Moved epmgp to a more appropriate place. The code there seems to follow the paper notation very closely, so that may as well stay as it is. But for the functions this package defines we should at the very least have types and docstrings. @aaronkl would you be able to help write those docstrings?


Unnecessary rows on "All containers" screen

Steps:

Open /clusters/all

Result:

Many empty rows are displayed with node status "Node is offline"

#56 created task for refactoring. as workaround was added filter


⚠ Samsung has degraded performance

In 2f4a8ba, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2453 ms

Resolved: Samsung performance has improved in 0de6065.