[Linear] Can't authorize extension while under VPN

Extension
https://github.com/raycast/extensions/tree/8af9388f217e31487aecdf1284e8718a619766d3/extensions/linear/
Description

Error:
TypeError: fetch failed

Object.processResponse:undici:6406:34
undici:6727:42
AsyncResource.runInAsyncScope:node:async_hooks:203:9

Steps To Reproduce

macos ventura macbook m1 max 16" 2021, raycast 1.44.0. cisco anyconnect corporate VPN
open create issue command. then click sign in with Linear. it opens browser, press Authorize Raycast.
error

Current Behaviour
It opens Raycast, for a moment everything is ok, then it shows error message something is wrong, then it shows blank page

Expected Behaviour
Normal error message is shown saying I can't authorize. And better, I'm not required to disable VPN to authorize extension.

also while trying to create issue while using VPN
TypeError: fetch failed
    at Object.processResponse (node:internal/deps/undici/undici:6406:34)
    at node:internal/deps/undici/undici:6727:42
    at node:internal/process/task_queues:140:7
    at AsyncResource.runInAsyncScope (node:async_hooks:203:9)
    at AsyncResource.runMicrotask (node:internal/process/task_queues:137:8)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Thanks for the issue. I'll take a look at that next week.

I tried reproducing it with a VPN (NordVPN) without any success. All worked fine on my end. Are you able to log in to Linear without the VPN?

@thomaslombart could you please provide me with request that Raycast is doing to login to Linear? I could ask if my VPN admins blocked domain or some parts of this request.

You can get the authorization link on the OAuth login screen:

The authentication process goes through a proxy located at https://linear.oauth-proxy.raycast.com/.
If you need more info on the API calls, you can check out the source code as well: https://github.com/raycast/extensions/blob/main/extensions/linear/src/api/oauth.ts

@lainisourgod did you find anything?


fix: 'TargetLanguage' has no exported member 'VISUALIZE'

The signature overwrite for targetName using TargetLanguage.VISUALIZE causes and error, because the enum value is stripped from the generate type declarations. We need to also strip the signature overwrite.
I manually checked the output to confirm. Not sure how else to test this.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

💚 All backports created successfully

Status
Branch
Result

✅
maintenance/v5.2

✅
maintenance/v5.1

✅
maintenance/v5.3

Questions ?
Please refer to the Backport tool documentation and see the Github Action logs for details


Clean installation on Ember 2.8 fails

I have a hard time starting a clean ember app just to test the library. Ember CLI 2.8 and Ember 2.8beta1
ember new dynform
ember install ember-cli-dynamic-forms
Installation fails, it seems the script stops after specifying Jquery 1.13. Doing a manual bower install --save jquery alpaca handlebars bootstrap lodash seems to get me going
ember g route index
ember g controller index
// controllers/index.js
import Ember from 'ember';

export default Ember.Controller.extend({
  schema: {
    "title":"User Feedback",
    "description":"What do you think about Alpaca?",
    "type":"object",
    "properties": {
      "name": {
        "type":"string",
        "title":"Name"
      },
      "feedback": {
        "type":"string",
        "title":"Feedback"
      },
      "ranking": {
        "type":"string",
        "title":"Ranking",
        "enum":['excellent','ok','so so']
      }
    }
  }
});

// templates/index.hbs
INDEX

{{dynamic-form schema=schema}}

The above code fails with an error from Alpaca: {"message":"A view was not specified and could not be automatically determined."}
I'd love to work with this plugin, but for now I'm stuck...

Ok, works now, I got the controller wrong. The schema object should contain keys ['schema', 'options', ...]
// controllers/index.js
import Ember from 'ember';

export default Ember.Controller.extend({
  formOptions: {
    schema: {
      "title":"User Feedback",
      "description":"What do you think about Alpaca?",
      "type":"object",
      "properties": {
        "name": {
          "type":"string",
          "title":"Name"
        },
        "feedback": {
          "type":"string",
          "title":"Feedback"
        },
        "ranking": {
          "type":"string",
          "title":"Ranking",
          "enum":['excellent','ok','so so']
        }
      }
    }
  },
});

// templates/index.hbs
INDEX

{{dynamic-form schema=formOptions}}

What is the reasoning for using the schema attribute name on the dynamic-form component? I misunderstood it's usage...

Hi @ssured, sorry for the confusion.  ember-cli-dynamic-forms was originally constructed as a wrapper to alpacajs, which in its form implementation provides an additional options object to provide annotations onto the json-schema, such as label values, event handling, field length, etc.
see http://www.alpacajs.org/tutorial.html for a basic example.
Given some good use cases for it we could split these into 2 attributes.  My assumption thus far is that these objects stay together.

Thanks for the info, to me it seemed there's more passed in the schema argument than just a schema. I was thinking about a more generic name, like config. I should have read the documentation in more detail.
What is still open is that the plugin doesn't install comfortably in cli 2.8 + ember 2.8, which is what this issue was originally about.

Thanks for reporting.  I'll look into it.  The alpaca plugin has a hard jquery dependency which has caused issue in the past, but I've been able to specify a resolution for it.  Maybe something changed recently.  I'll have a look.

@ssured see if version 0.2.3 gets you a full install.  There was a left-pad type issue with one of alpaca's dependencies, causing the bower install process to fail.

@toddjordan thanks for your input so far, I'm experiencing the exact same issue with version 0.2.3 and ember-cli 2.8.0. Any suggestions for debugging I could perform?

Hi @hcderaad I will check on it today, and let you know what I find.   maybe/hopefully we can axe that dependency 👍

yeah, there seems to be a problem again with dependencies :-(  I'm working on it today and will let you know when there's a new release to try.

Quick update:
Looked at this a little yesterday.
There seems to be an issue with the latest ember cli where a dependency conflict prevents the default blueprint from running.
jquery is still an issue (the combination of bower packages we use (alpaca, bootstrap, etc) use conflicting versions.  I'm going to try out some things today to see if I can find a workaround.

@hcderaad I'm at a slight impass here.   This is what I've discovered
There's a guaranteed jquery conflict in newer versions of ember with newer versions of alpaca.  This is causing the default blueprint to halt execution and require the user to enter a preferred jquery version.  The problem is that once the user selects a version, the default blueprint does not finish running, and thus no bower dependencies are added to your project.  The workaround for now is manually add the necessary packages to bower which is what the default blueprint should do).  You'll want your bower.json to look something like this.
{
  "name": "my-proj",
  "dependencies": {
    "ember": "~2.8.0",
    "ember-cli-shims": "0.1.1",
    "jquery": "~3.1.1",
    "bootstrap": "3.3.2",
    "lodash": "~4.6.1",
    "alpaca": "1.5.22"
  },
  "resolutions": {
    "jquery": ">= 2.1.0",
    "tinycolor": "1.3.0",
    "bootstrap": ">= 3.0.1"
  }
}

ember and ember-cli-shims are on every ember project.  The rest of the above packages are required to run ember-cli-dynamic forms.
Its the need for the resolutions section is what's choking the ember install atm.  I'm going to pursue a solution with the ember-cli team.
Thanks, and sorry about the inconvenience.

opened https://github.com/ember-cli/ember-cli/issues/6334

@toddjordan Thanks for following this up, if there is anything to test/debug please let me know!

@toddjordan Would perhaps this pull request provide some insight into the issue? https://github.com/ember-cli/ember-cli/pull/6319

@toddjordan sorry to bother you again, but has there been any update on this? As this issue is preventing me to use ember-cli-dynamic-forms at all, using alpacajs with a native bower install seems to work well enough. The workaround of manually setting the required bower.json resolutions does not result in a working installation. The installation breaks when choosing the appropriate jquery version in the ember-cli-dynamic-forms installation and this results in a corrupted broccoli build attempt. Any other avenues to pursue here perhaps? Thanks for your efforts!

It doesn't seem like 6319 is the same thing.  that is addAddonToPackage.  This one is bower and only happens when there is some version conflict.
I'm really sorry for your struggles here.  I'm surprised you can't get working by installing the required bower libraries.  When I do it myself with a new project things seem to work.  I'll record the steps I use... maybe that would help.
ember new test-proj
cd test-proj
ember install ember-cli-dynamic-forms

now pick jquery >= 2.1.0
then go to bower.json and set the following
{
  "name": "my-proj",
  "dependencies": {
    "ember": "~2.8.0",
    "ember-cli-shims": "0.1.1",
    "jquery": "~3.1.1",
    "bootstrap": "3.3.2",
    "lodash": "~4.6.1",
    "alpaca": "1.5.22"
  },
  "resolutions": {
    "jquery": "3.1.1",
    "tinycolor": "1.3.0",
    "bootstrap": "3.3.2"
  }
}

then bower install
Things should work from there I think.
As far as the issue goes I'm at a sort of impasse.  I've gotten no response on the issue I created with ember-cli.  The only 2 options I see are as follows:

remove the alpaca dependency
fork alpaca and give it more flexible versioning

1 has started and its the most work, but its where I'd like to see the project go long term, I have just been consumed with other things and haven't given it proper time.
2 seems simpler as a short term fix, but I'm unsure what impact it will have on runtime execution until I try it out.

in thinking about it a little more, it might work to specify the bower resolutions before installing the addon (maybe with >='s).  I might try that later.  Again, that's just another workaround until we get something fixed.

Installing manually works up to a certain extent, but I'm afraid the Alpaca dependency on older versions of (this time) bootstrap is creating other conflict issues (e.g. not being able to run ember-bootstrap). I'm afraid I don't see many options left but to look for another form generator. Which is a pity, as I really liked working with alpacajs in the past, especially because of the json schema driven form creation (combined with server side json schema driven validation a breeze).... Anyway, many thanks for your efforts! If you have any other ideas, please do let me know, I'd be very happy to test them!

ok, really sorry.  I'll be looking at it some in the coming weeks.  If I get a response/help from the ember-cli team or come up with a solution on this side for the versioning issues, I will update this issue and cc you.
Thanks for the feedback and I appreciate you trying it out.

Small update... working with ember-cli team to get a fix for this in the coming week or so.

Any update we have stuck on the same issue as well.

@kprasad73 I've created a PR to ember-cli here: https://github.com/ember-cli/console-ui/pull/7 see the issue for history: https://github.com/ember-cli/ember-cli/issues/6334
It got push back a couple of months (sorry), as I got wrapped up in other stuff.  I will hopefully show up with an upcoming release of ember-cli.  Until then and for past (current) versions of cli, there is a workaround (see my comment on Oct 20).  Essentially its manually adding the bower packages to bower.json  and running bower install.

Thanks @toddjordan, followed the steps outlined in your comments. However i am now getting Uncaught Error: Could not find module ember imported from .....
Googling to find a solution to the above error. Any pointers will help

@kprasad73 What version of npm are you using, and what version of ember-cli?

npm version 3.10.8
ember-cli version 2.7.0
node version 6.9.1
OS is win32 x64
Appreciate all the help.

Good news.  Fix got merged into<EMAIL_ADDRESS>(a dependency of ember-cli) upcoming ember-cli should have the fix.

Fix coming in ember cli 2.11

Verified fixed on 2.11 beta 1.  For 2.10 and below, will need to use the workaround above.

Just a side note:  You will get prompted for conflicts, since alpaca and ember use different versions of dependencies, and alpaca seems to like to pin versions :-(   Just take the last option of each list when prompted and things should work out.

lol, one last thing to mention (maybe)... you'll need the latest version of this addon (0.4.0) to work with 2.11, because of some removed apis


"All Permissions" is not displayed for sub-tenant admin user

Describe the issue:
"All Permissions" is not displayed under permission list for sub-tenant admin until token is generated for the first time.
How to reproduce:

Setup components mentioned in [Environment information]
Login to management console of IS with primary tenant admin
Create a new sub-tenant test.com
Login to management console of IS with sub-tenant admin
While creating new role, under permission list only "Admin Permissions" is listed as per the below screenshot1.png

Login to store of WSO2APIM with sub-tenant admin.
Generate access token for Production.
Again login to management console of IS with sub-tenant admin
While creating new role, under permission list "All Permissions" is listed as per the below screenshot2.png

Expected behavior:
"All Permissions" should be displayed (as displayed for primary tenant admin) regardless of the access token generation.
Environment information:

WSO2IS as Key Manager 5.7.0, WSO2APIM 2.6.0
OS: Linux
Database: MySQL 5.7
Userstore: LDAP

Going forward role permissions will be managed by the Console app. In the console, the permission tree is rendered in a unified way for all the tenants.

Going forward role permissions will be managed by the Console app. In the console, the permission tree is rendered in a unified way for all the tenants.

Going forward role permissions will be managed by the Console app. In the console, the permission tree is rendered in a unified way for all the tenants.

Hi @ayshsandu ,
Thank you for sharing the information. I have understood it.
I have query regarding displaying permission tree with and without token generation:
Case1 : Permission Tree before token generation
Only Admin Permissions is displayed.
Case 2 : Permission Tree after token generation
In addition to Admin Permissions, Applications and their parent All Permissions is also displayed. As mentioned in screenshots here
As per my understanding, the expected behaviour is All Permissions should be displayed regardless of token generation.
Please let me know if my understanding regarding expected behaviour is not correct.

Going forward role permissions will be managed by the Console app. In the console, the permission tree is rendered in a unified way for all the tenants.

Hi @ayshsandu ,
Thank you for sharing the information. I have understood it.
I have query regarding displaying permission tree with and without token generation:
Case1 : Permission Tree before token generation
Only Admin Permissions is displayed.
Case 2 : Permission Tree after token generation
In addition to Admin Permissions, Applications and their parent All Permissions is also displayed. As mentioned in screenshots here
As per my understanding, the expected behaviour is All Permissions should be displayed regardless of token generation.
Please let me know if my understanding regarding expected behaviour is not correct.


Tutorial stops at chapter 4 "Services"

The current version of the "Tutorial: The Tour of Heroes" ends at chapter 4 "Services" somewhere halfway the implementation of the Vision.
You can manually go to chapter 5 by changing the URL to https://angular.io/docs/ts/latest/tutorial/toh-pt5.html but this page contains many error messages like:
BAD FILENAME: ../../../_fragments/toh-4/ts-snippets/app/app.component.pt4.ts.md   Current path: docs,ts,latest,tutorial,toh-pt5 PathToDocs: ../../../

We know! Tutorial is a work in progress. Part 5 is a sketch and not ready to be seen ... which is why didn't link to it.
It will be ready soon. Please hang in there.
Closing this issue for now. Reopen if we don't make good on the promise of part 5 soon.

Where could see the sketch?

Can we reopen this issue please?

Ok. But please ... no more complaints about ToH part 5 being "broken". It was never published so it can't be broken. It does need to get finished.

Fixed by PR #844.

It's fixed now.


[Bug]: Cartesian plotting axes not displaying properly

What happened?
Plotting with dipole seems to work okay, but on a Cartesian grid I get stuff like this:

Relevant log output
No response

I suspect the data may need to be transposed prior to plotting?

I'll still having this issue as of now.


[stable/minio] allow to set lifecycle to the default bucket

It would be great to be able to configure a lifecycle link the "defaultBucket".

Feel free to send PRs to https://github.com/minio/charts we have moved.


Add enchants for Enchant Parsing

Repo file for all enchants for Enchant Parsing (will make pr in mod soon). Includes the new ones SBA doesn't have

bruh

bruh

why bruh?

Dont see why this is needed, we already have enchant colors in NEU, so good and max level is not needed.

Dont see why this is needed, we already have enchant colors in NEU, so good and max level is not needed.

It's based on SBA's enchant parsing which colors them based on how good the level of the enchant is, which is different for man y enchants. With NEU you can only color them if they are above, below or equal to a level that matches a certain regex, i.e., 5 isn't max for Power but it is for Charm for example, with NEU you would end up having those be the same color, unless you manually recolor every enchant manually or try to group some with regex rules which is not really viable.
I and many others prefer SBA method of coloring the enchants. If hanni and others really don't want this feature and don't see the need then sure I guess I'll remove it. But I right now I see it as sufficiently different from NEU's one (and better IMO)

Or you can just download a preset, which already does that for you. This is just duping a feature and not making it better, but thats does not matter for this pr. I just dislike it really hard

Why go through the effort of finding a preset that probably has countless rules which someone may or may not have yet made. Then when you want to change a color of 1 or more enchants you have to scroll through to find them then change the color of each individually, and possibly the level of said enchant if Hypixel decide to add a new level to an enchant.
Whereas with this one you can just change the color of the max level enchants, great level enchants, good level enchants, and poor level enchants with relative ease. I wouldn't say its duping a feature, rather a (better) alternative. Also, seriously? You haven't seen one person that prefers SBA's one over NEU? Let me be the first I guess.

I think we should wait for hannibals opinion in this, for me its a FAR FAR worse alternative. Instead i would just add a button to neu to maxout enchants.

While i think it's true that most users only care for hyper maxed/super good /descent/bad enchantments color and ignoring ultimate enchants, i do like the endless possibilities of neu's custom settings. I don't know how useful a preset would be, since it feels a bit static when changes happen.

Doesn't sba's enchant parsing change how the enchants are sorted or something too?
off

compress

normal

expand

You should add Pesterminator btw, in case this ends up getting merged. I guess maybe good level can be like 3, I saw a lot of people with that. Max is 5.

You should add Pesterminator btw, in case this ends up getting merged. I guess maybe good level can be like 3, I saw a lot of people with that. Max is 5.

Yeah, there have been a few enchants that have been added to the game since this PR, but I haven't been able to get around to it since I have been unbelievably busy with uni work, if I manage to find time in the next week I'll probably add the missing ones. Also thanks for the suggestion on the good level since I have no idea about any of the new enchants 😅.

Sba's enchant parsing does more than just change enchant colors-

Correct, at the moment my enchant parsing forces compressed at the moment (3 enchants a line), the feature is not as exhaustive as sba's (yet). I've not been able to go over the code for my enchant parsing since I made the pull request, since as mentioned above, I have been extremely busy, and I wanted to re-write some of the code for it since I feel it could be improved a bit. With the exception of waiting until the missing enchants have been added to this PR, it should be (the PR on the main repo) in a state to be merged. I do plan to add the three options of formatting like in sba but don't have the time right now. (Also I think your comment may have been more relevant on the main repo than here)


Merge with emirpasic/gods ?

A few days ago i found https://github.com/emirpasic/gods.
Go Data Structures. Tags: Containers, Sets, Lists, Stacks, Maps, Trees, HashSet, TreeSet, ArrayList, SinglyLinkedList, DoublyLinkedList, LinkedListStack, ArrayStack, HashMap, TreeMap, RedBlackTree, BinaryHeap, Comparator, Sort

Maybe it make sense to merge both together? Or to benefit from each other?

No response since ~6 month. So i think this is not wanted

@andygrunwald sorry about the delay. gods does look promising, but merging the projects would be a lot of work due to conflicting designs. I'll bring this up for discussion with my team. I do like the interfaces defined in gods and it may be worth satisfying those interfaces with go-datastructures implementations.
For what it's worth, Workiva/go-datastructures is currently ~5 times larger than emirpasic/gods:
wc -l $(git ls-files | grep "go$")

gods has 5,908 lines, while go-datastructures has 26,046.

With merging i mean "merge both projects together". In what direction, doesn`t matter.
But maybe a link to gods might be cool to know what projects in this area exists as well.


Change stable_identifier_type from String to Enum

There is currently only one Stable ID identifier type: FREG_SNR. This should be an enum so that clients do not have to hard-code this value.

After discussion with @mmwinther I'm closing this PR. The ssb-datadoc-model should try to be as generic as possible and should not enforce foo many Statistics Norway specific details.


Fix filter by date start of today filter + Translations

Previously, the filter caused no files to sync unless some files were set in the future
Now, it will sync all files modified since TODAY

Hi,
This makes the Sync from today work properly and not only on files eventually modified start of tomorrow (in the future)
Thank you for the new filters, they are useful to have
Best regards

The >= part is not needed, sorry. Just noticed the time was properly reset to 00:00:00 on filter date
I dont have access to git right now to change it. Feel free to merge only the translation part
Best regards


xcbuild: fix missing library for PlistBuddy

Executing the PlistBuddy binary currently aborts on Darwin, because a dynamically linked library is missing:
dyld: Library not loaded: /nix/store/144qrs12839p8a0dj9cyvdhxf2s7dvjs-xcbuild-0.1.2-pre/lib/liblinenoise.dylib
  Referenced from: /nix/store/144qrs12839p8a0dj9cyvdhxf2s7dvjs-xcbuild-0.1.2-pre/bin/PlistBuddy
  Reason: image not found
fish: Job 1, '/nix/store/144qrs12839p8a0dj9cy…' terminated by signal SIGABRT (Abbruch)

The reason is that liblinenoise.dylib is introduced to the build as third party code, but the resulting library is not installed. I fixed this by manually installing the library. Execution of PlistBuddy works as expected now.
Additional Remarks

The package currently does not build on Linux, so I did not test PlistBuddy there.
There is a TODO comment above the postInstall attribute asking for a cmake fix for the installation in $out/usr. I looked into this and unfortunately, the /usr part is hardcoded in multiple CMakeLists.txt throughout the source. I feel that patching those is more brittle than the solution currently in use, so I would vote to remove this comment here. I can amend the PR if needed.

Things done

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

[ ] NixOS
[x] 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"
[ ] 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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117967 at b28953e0 run on aarch64-linux 1

2 packages failed to build:

xcbuild (xcodebuild)
xcodebuild6

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

/rebase staging

/rebase staging

The actions bot is failing with this:
 ! [remote rejected]         patch-xcbuild -> patch-xcbuild (refusing to allow a GitHub App to create or update workflow `.github/workflows/editorconfig.yml` without `workflows` permission)
error: failed to push some refs to 'https://github.com/mroi/nixpkgs.git'
Error: Process completed with exit code 1.

I am unfamiliar with GitHub actions, so I am not sure how to fix that to cause the bot to succeed. Should I just retarget this PR manually?

I am unfamiliar with GitHub actions, so I am not sure how to fix that to cause the bot to succeed. Should I just retarget this PR manually?

I think you have disabled that maintainers can push into your branch. You can retarget the branch manually but please be careful to not ping a 100 people.

I just changed the base branch of this PR to staging. I hope this was the right course of action here.
(I did not knowingly configure any restrictions on branch access. Everything looks alright to me.)

I just changed the base branch of this PR to staging. I hope this was the right course of action here.

Seemed to work.

🤔 this is not new, right?
find: File system loop detected; ‘../results/xcbuild/usr’ is part of the same file system loop as ‘../results/xcbuild’.
find: File system loop detected; ‘../results/xcbuild/Platforms/MacOSX.platform/usr’ is part of the same file system loop as ‘../results/xcbuild/Platforms/MacOSX.platform’.
find: File system loop detected; ‘../results/xcbuild/Toolchains/XcodeDefault.xctoolchain/usr’ is part of the same file system loop as ‘../results/xcbuild/Toolchains/XcodeDefault.xctoolchain’.

I get these symlink loops with current Nixpkgs master as well. It seems to be independent of my patch.
The symlinks are explicitly created here (commit):
https://github.com/NixOS/nixpkgs/blob/776241d186de9aed6465882f9f2c6102709569ee/pkgs/development/tools/xcbuild/platforms.nix#L297
https://github.com/NixOS/nixpkgs/blob/776241d186de9aed6465882f9f2c6102709569ee/pkgs/development/tools/xcbuild/toolchains.nix#L29
https://github.com/NixOS/nixpkgs/blob/776241d186de9aed6465882f9f2c6102709569ee/pkgs/development/tools/xcbuild/wrapper.nix#L102
I would say they are there on purpose for tools expecting /usr in the path.


Complex search

Can anyone please take a look at my question at S.O?
http://stackoverflow.com/questions/31865876/complicated-many-to-many-relationship-in-dropdown-search-using-angularui-select

I got it working :)


Frozen hashes/sets

Is your feature request related to a problem? Please describe.
It is more a discussion, than a real issue or feature request. I'm looking for quite an unusual thing - an ability to freeze a hash or a set in some shared memory piece, so other processes can access this data structure from shared memory concurrently but with no ability to change data. So I'm looking for any hints in this direction about the potential design. I'd be also fine if that will not come to the upstream as a rarely needed feature but in my case it can significantly reduce memory footprint especially in highly loaded systems. And since no modifications are possible, there is no need in locking or other consistency support - processes can just read data from the shared memory.
Describe the solution you'd like
I would like to have an interface that stores data in a given shared memory with an ability to restore data afterwards. The most simple approach is to copy the underlying vector into shared memory, and create a structure directly in shared memory with placement new and set the storage pointer to the vector allocated/copied previously. This is quite a low level approach, so I'm wondering if there is something better.
Describe alternatives you've considered
In my case, the alternative is to have a separate hash table for each process which is just a waste of memory.
P.S. please feel free to close if this is not relevant to the project.

If I understand you right, you want to allocate shared memory data somehow and then have the map inside this shared block of data? This sounds like a job for a custom allocator. If your compiler supports it, I think the easiest way to do this would be to implement an std::pmr::memory_resource that actually allocates a block of data in shared memory and then gives out pointers to this shared memory as needed.
Have a look at test/unit/pmr.cpp. Here is a short example that will use logging_memory_resource and log exactly two allocations, one for the vector, and one for the indexing struct:
    auto logging_resource = logging_memory_resource();
    auto map = ankerl::unordered_dense::pmr::map<uint64_t, uint64_t>(&logging_resource);
    map.reserve(100);
    for (uint64_t i=0; i<100; ++i) {
        map.try_emplace(i, i);
    }

You could modify logging_memory_resource to instead provide pointers to shared memory.

You could also have a look at

metall: provides an allocator on memory mapped files. You can open the memory mapped file the in another process as well.
boost interpocess (provides allocators that work -- among other types -- with shared memory)

Both solutions have a downside though: You would need to replace the std::vector used by ankerl::unordered_dense internally with boost::container::vector.
The container types from the standard library (at least libstdc++) do not comply to custom (fancy) pointer types defined by allocators and use raw pointers instead. Both, the metall allocator and the boost interprocess allocator use offset pointers not raw pointers. So it would need some tinkering :)

Both solutions have a downside though: You would need to replace the std::vector used by ankerl::unordered_dense internally with boost::container::vector.

I'm working on a branch where it is possible to customize the container with a simple template argument. It's not yet finished though.

I'm working on a branch where it is possible to customize the container with a simple template argument. It's not yet finished though.

Great... I would not have dared to ask for this :)

In #20 I just merged the option to use custom containers. The Allocator template argument is now AllocatorOrContainer and this accepts a container type now too. E.g. this works:
using Map = ankerl::unordered_dense::map<
    int,
    std::string,
    ankerl::unordered_dense::hash<int>,
    std::equal_to<int>,
    std::deque<std::pair<int, std::string>>>;

I've got a test in custom_container_boost.cpp that makes use of shared memory. This little example works on my machine, but I didn't thoroughly test it.


Fix Node.js SDK generation

The Pulumi Node.js SDK bumped the TypeScript version from 3.7.3 to 3.8.3 to be able to use OpenTelemetry. Unfortunately, this has broken the SDK doc generation.
This change addresses the problem by upgrading TypeDoc to a version that supports TypeScript 3.8.3 and fixing our tscdocgen tool to handle the new output from the upgraded version of TypeDoc.
Fixes https://github.com/pulumi/pulumi/issues/10787

Mildly concerned about types like Lifted, Unwrap, etc., leaking into our docs. Really not anything end users need to be aware of, so if we can't hide them, I wonder if we could note that they're types used internally to provide a smooth experience using Pulumi TypeScript?

I agree that these would ideally be hidden, but prefer to consider that in a follow-up since we were already including them in the docs currently. The handling of the new types (conditional and infer) from the new version of TypeDoc was done to prevent failing on unrecognized types. I guess instead of actually providing an implementation for these, I could have tried to make it so that they rendered as they did previously (perhaps by just returning an empty string).
In any case, we should be cautious about investing more in improving our tscdocgen tool. We originally used this to generate Node.js docs for all our providers, and the output is very oriented towards that (i.e. there's a Resources header, that's meant to show all the resources in a given module; which doesn't really fit with the docs for our core SDK, which doesn't have resources per se other than the base Resource/CustomResource/ComponentResource classes). We've long since moved away from using this tool for our provider docs (we have another tool that generates these for the registry, and emits declarations for all languages).
We have a similar problem with Python SDK docs, which are awful.
Rather investing in our bespoke tools for Node and Python, we should consider moving to standard doc generation approach like we do for .NET, Go, and Java.


Passing project properly in BQSource

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python 3.8 PostCommit

r: @kamilwu

Run BigQueryIO Read Performance Test Python

FYI @youngoli I am a little concerned that this may be a issue that we should cherrypick for 2.25....

oops my bad, not @youngoli  but @robinyqiu

Is this fixing the performance regression mentioned in the other PR? Could you please create a JIRA for this?


Add macOS universal builds to Luci

Related: https://github.com/flutter/flutter/issues/60113, https://github.com/flutter/flutter/issues/69221
Engine is now ready to be cross compiled to darwin-arm64. gn --mac and gn --mac --mac-cpu=arm64 will produce x64 and arm64 builds of engine and desktop artifacts.
We need a script to lipo macOS framework (similar to sky/tools/create_ios_framework.py) and desktop artifacts.
Also a build recipe for Luci.

cc @jmagman, @jonahwilliams

\cc @gw280 who has context on how the engine builders artifacts will be broken up to avoid timing out w/o goma

See https://github.com/flutter/flutter/issues/81855

I'll ensure this is reflected in the dependency graph


How to inference on GPU?

❓ Questions and Help
Hi.
I have trained a Model with Trainer.fit(). Now I want to load the checkpoint at another place and preform inference. But I have no idea how to inference on GPU.  Where could I assign a GPU for my inference just like assigning a GPU before training:
trainer = pl.Trainer(max_epochs = cfg['n_epochs'],
                            callbacks=[checkpoint_monitor, lr_monitor],
                            gpus=1)

My inference code is (simplified version):
model = plModel.load_from_checkpoint('lightning_logs/version_0/checkpoints/model_epoch=17_val_loss=95.99.ckpt')
model.eval()
test_loader =test_dataloader()

correct_count = 0
nsample = len(test_loader)
for i, data in enumerate(test_loader):
    output = model(data['x'])
    label = data['label'].data.cpu().numpy()
    pred = nn.functional.softmax(output, dim=1)
    pred = np.argmax(pred.data.cpu().numpy(), axis = 1)
    correct_count += np.sum(pred == label)
accuracy = correct_count / nsample

What's your environment?

OS: [ Linux,]
Packaging [pip]
Version [1.1.0]

Hey, in case you didn't know, we have a forum here: https://forums.pytorchlightning.ai/ :)
You can perform the inference like this (using your code sample):
trainer = Trainer(gpus=1, ...)
model = plModel.load_from_checkpoint('lightning_logs/version_0/checkpoints/model_epoch=17_val_loss=95.99.ckpt')
# not needed
# model.eval()

trainer.test(model, test_dataloaders)

# move the rest to the LightningModule:

class PLModel:

def __init__(self, ...)
   self.accuracy = pl.metrics.Accuracy()

def test_step(self, batch, batch_idx)
    data = batch
    correct_count = 0
    nsample = len(test_loader)
    output = self(data['x'])
    label = data['label'].data.cpu().numpy()  # optimize this and use torch operations on the gpu instead of numpy
    # use the Metrics package, you can directly feed it logits
    # pred = nn.functional.softmax(output, dim=1)
    # pred = np.argmax(pred.data.cpu().numpy(), axis = 1)
    self.accuracy(output, label)

def test_epoch_end(self, outputs):
    # log or print
    self.log('test_accuracy', self.accuracy.compute())

caution: brain-compiled code
References:

https://pytorch-lightning.readthedocs.io/en/stable/metrics.html

Hey, in case you didn't know, we have a forum here: https://forums.pytorchlightning.ai/ :)
You can perform the inference like this (using your code sample):
trainer = Trainer(gpus=1, ...)
model = plModel.load_from_checkpoint('lightning_logs/version_0/checkpoints/model_epoch=17_val_loss=95.99.ckpt')
# not needed
# model.eval()

trainer.test(model, test_dataloaders)

# move the rest to the LightningModule:

class PLModel:

def __init__(self, ...)
   self.accuracy = pl.metrics.Accuracy()

def test_step(self, batch, batch_idx)
    data = batch
    correct_count = 0
    nsample = len(test_loader)
    output = self(data['x'])
    label = data['label'].data.cpu().numpy()  # optimize this and use torch operations on the gpu instead of numpy

    # pred = nn.functional.softmax(output, dim=1)
    # pred = np.argmax(pred.data.cpu().numpy(), axis = 1)

    # use the Metrics [1] package instead, you can directly feed it logits
    # and it correctly works across multiple gpus should you need that.
    self.accuracy(output, label)

def test_epoch_end(self, outputs):
    # log or print
    self.log('test_accuracy', self.accuracy.compute())

caution: brain-compiled code
References:

[1] https://pytorch-lightning.readthedocs.io/en/stable/metrics.html

Thx so much. First time know the forum.

If you want direct access, it can be done directly using the forward pass as follows. (The way it is done in PyTorch.) This gives you direct access to the variables.
model  = YourLightningModule.load_from_checkpoint(r"path/to/checkout.ckpt")
model.to(device)
data.to(device)
with torch.no_grad():
    out = model(data)

You can loop over a dataloader if you want.

If you want direct access, it can be done directly using the forward pass as follows. (The way it is done in PyTorch.) This gives you direct access to the variables.
model  = YourLightningModule.load_from_checkpoint(r"path/to/checkout.ckpt")
model.to(device)
data.to(device)
with torch.no_grad():
    out = model(data)

You can loop over a dataloader if you want.


yp-spur: 1.22.2-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository yp-spur to 1.22.2-1:

upstream repository: https://github.com/openspur/yp-spur.git
release repository: https://github.com/openspur/yp-spur-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.22.0-1

ypspur
* Fix system time jump threshold (#200 <https://github.com/openspur/yp-spur/issues/200>)
* Contributors: Atsushi Watanabe

Holding for Noetic sync


Use WebWorker?

If we can't get rid of the long blocking due to copying procedures (which are sadly necessary for Uint8Array), it might make sense to use a dedicated WebWorker for this.

Yup. I think we can close this since it doesn't require any modification to the code.

@beaulac nice to hear that other people than me are using the library :slightly_smiling_face:


Create constants for parameters like freshness period

          Maybe we could create a constant for the freshness period to make it clearer why that value has been chosen?

Originally posted by @JKRhb in https://github.com/hsel-netsys/iceflow/pull/18#discussion_r1295915106

Resolved by #27.


refactor: Easily change complex logic

Removed complex branching statements from getIdentifierColumns()

No need to check that "joins" size is 1

Changed the logic of "handleSelect()" to be more readable

The code worsened in three places. Why fix whats not broken?


(update) Install watchman w/ --HEAD to avoid error

Updated the docs for installing watchman with --HEAD per @wez suggestion in this issue thread https://github.com/facebook/react-native/issues/239

We have it in the troubleshooting page. @wez is it long before you update brew recipe?

I'll probably submit the PR to brew this weekend (I want to get in a change that will remove a launchd subscription when you watchman shutdown-server) and it will take however long for the brew folks to review and ship.
At that point we should probably suggest that the minimum recommended watchman version to use is 3.1, so maybe we can tweak this PR to indicate that instead of the HEAD install?

Cool, I'll pull this now and hope that master on watchman will continue to be stable until we publish 3.1.

Cool, thanks!


[csharp,iOS] Fix callbacks on AOT

When using AOT mode on Mono one can't use instance methods for callbacks, the compiler needs the MonoPInvokeCallback attribute on a static method to know how to get back to the managed world.
This worked fine without the change in JIT mode.
I not sure if we should use a MONO flag for this stuff as this could be needed for other usage that not only on iOS.
The adicional change is that one should as good practice call Dispose on the YogaNode when using callbacks to make sure we remove the handler from our dictionary, i was trying to write a test for this.. but i saw TestMeasureFuncWithDestructor was already there, but it doesn't seem correct to me, can you check @splhack.

I'll look into this.

Ok after internal feedback from Alex Corrado i got a new implementation for the fix using the native ref context of Yoga.

The test for the Parent destructor is failing for me on AOT .. still looking at that one.

can you take a look at #387 ?

@splhack It looks like a more complicated way of doing the same thing? I do see one improvement: using a weak GCHandle and only allocating one of them.
Also note that neither PR appears to ever free the GCHandle

Ah, I haven't seen the updated diff.

revised in #388

Could you rebase this onto the current master too? I'll merge this and #388

It's rebased on top of master..


Components menu pushed down

On component pages, the mobile components menu is getting pushed down below the first heading.

I had a fun morning trying to find the source of this bug. It's due to #571.
I feel this should be done in the site's templates rather than the page-navigation component. Page-navigation (and content sites in general) should not have a monopoly on the sidebar. People will need to use the sidebar for other things. This is the kind of "Wait, what just happened?" function that's really going to trip people up when they try to use our Design System.
I can make this change but wanted to check and see if I'm missing anything. cc @aaronhans @kkoryaka

Yeah that makes sense, better not to have that logic inside the component. Thanks @xjensen

Yes, make sense. So, should we close this #571 issue for these reasons?

Confirmed that component menu is ABOVE now as expected. Pixel 3a chrome:


MaxListenersExceededWarning when running many watches

I run a handful of watches and I get these warnings:
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 unpipe listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 drain listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 finish listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 unpipe listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 drain listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added. Use emitter.setMaxListeners() to increase limit
(node:15779) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 finish listeners added. Use emitter.setMaxListeners() to increase limit

When this happens, it doesn't seem to detect changes to files.  It's possible this is actually a nodemon issue.

Upgrading nodemon seems to have helped in terms of detecting changes.  However, it still prints these warnings prior to launching any nodemon process.  Maybe just need to call emitter.setMaxListeners() somewhere?
To reproduce, just set up a bunch of watch configs.

see #81, there is a release coming soon that will allow this setting


x/pkgsite: time package displayed incorrectly

Current:

The formats and 002 are space-padded and zero-padded three-character day of year...

Desired:

The formats __2 and 002 are space-padded and zero-padded three-character day of year...

See https://pkg.go.dev/time#pkg-constants

The text in the file is "The formats __2 and 002 are space-padded and zero-padded".  However, the display at https://pkg.go.dev/time is "The formats and 002 are space-padded and zero-padded".  So somehow the pkgsite code is droppig the "__2".  It's not there in the HTML source ("The formats  and 002 are space-padded and zero-padded").

CC @jamalc

It is fixed on master so this should resolve with the next release.


Rename API functions to use snake_case instead of smushcase

E.g. get_mutable_version(x) instead of getmutableversion(x).
Also, in that specific case, maybe just mutable_version(x) is better?

+1 to mutable_version

Closing this! #11


Changing $rfs-rem-value doesn't show any effect

I tried to change the value of $rfs-rem-value but 1rem is still equal to 16px but what I want is to change the value of 1rem to 13px.

Hi @JeromeDeLeon,
Make sure sure to define $rfs-rem-value before importing the mixin, this will probably solve your issue.

I used BS4.3 and I found out that it uses rfs under the hood but changing it doesn't trigger any effect.
$rfs-rem-value: 13;

@import "/path/to/bootstrap"

Did I do anything wrong? I found similar issue and followed it by changing the font-size in HTML to atleast 88%.

#102

$rfs-rem-value is only used in the calculations to convert from px to rem or the other way around.
Because all font-sizes in Bootstrap are passed in rem to the mixin, you won't really notice any difference.
However if you use for example @include font-size(24px) in your custom css, you'll notice that the font size is still rendered in rem, but the calculations are based on a multiplier of 13 (for your case) so that the font size is still 24px for larger viewports.
So codewise:
In the first example we don't adjust $rfs-rem-value:

64px won't render as 64px for large viewports, but will rescale
The calculations for the responsive part will be slightly incorrect, you'll notice a small font size bump when you resize around the $rfs-breakpoint (1200px)
https://codepen.io/MartijnCuppens/pen/eYOoxRp?editors=1100

// Use default config
@import "path/to/rfs.scss";

html {
  font-size: 81.25%; // 13/16
}

.title {
  // This font-size will be 52px (81.25% of 64px) for larger viewports and will rescale on smaller screens:
  @include font-size(64px);
}

.title-2 {
  // This font-size will be 52px (81.25% of 64px) for larger viewports and will rescale on smaller screens:
  @include font-size(4rem);
}

In the second example we change $rfs-rem-value:

64px will render as 64px for large viewports
We won't have the font-size bump explained above.
Demo: https://codepen.io/MartijnCuppens/pen/ExYJrmy?editors=1100

$rfs-rem-value: 13;

@import "path/to/rfs.scss";

html {
  font-size: 81.25%; // 13/16
}

.title {
  // This font-size will be 64px for larger viewports and will rescale on smaller screens:
  @include font-size(64px);
}

.title-2 {
  // This font-size will be 52px (81.25% of 64px) for larger viewports and will rescale on smaller screens:
  @include font-size(4rem);
}

It can be quite hard to understand all this, I had to double check all this when I  was typing this comment, but in short:

If you change the font size of html, you should change $rfs-rem-value accordingly for the correct responsive calculations
You'll only notice a difference for large viewports if the font-sizes are called in px. Bootstrap has set all there font sizes in rem, so you probably won't spot the difference in that case.

Thanks for detailed explanation. I did followed the solution that was mention on the reference and it worked perfectly.


Wrong diff shown when adding Bootstrap

In this section of the guide, we add Bootstrap to our index.stache file, but the diff shows us adding the import for the view model instead of importing Bootstrap.

This is fixed in major.


'gopass binary cat' error with Jenkins pipeline

Summary
I'm trying to use gopass inside a Jenkins Pipeline and I get an error (No public key) when I tried to read the content of a file (a token) with the gopass binary cat test-gopass/foo/bar.
The error I've got is the following:
[...]
+ gopass binary cat test-gopass/foo/bar
gpg: B5B4BE68301B70047EB8FE3B93CB409BD2322BD7: skipped: No public key
gpg: [stdin]: encryption failed: No public key
2019/01/21 23:40:13 Failed to encrypt
[...]

Other details:

The command is executed in a Jenkins slave running as a container.
Running the same command on my system works fine.
gopass show test-gopass/foo/bar  works fine and displays the base64 encoded data.
Runing the command in debug level gives the following output:

[...]
+ GOPASS_DEBUG=true
+ gopass binary cat test-gopass/foo/bar
[DEBUG] Trying to load config from /home/jenkins/.config/gopass/config.yml
[...]
[DEBUG] fs.Get(.gpg-id) - /home/jenkins/test-gopass/.gpg-id
[DEBUG] gpg.listKeys: /usr/bin/gpg [/usr/bin/gpg --with-colons --with-fingerprint --fixed-list-mode --list-secret-keys B5B4BE68301B70047EB8FE3B93CB409BD2322BD7]
[DEBUG] gpg.listKeys: /usr/bin/gpg [/usr/bin/gpg --with-colons --with-fingerprint --fixed-list-mode --list-public-keys B5B4BE68301B70047EB8FE3B93CB409BD2322BD7]
[DEBUG] gpg.listKeys: /usr/bin/gpg [/usr/bin/gpg --with-colons --with-fingerprint --fixed-list-mode --list-secret-keys B5B4BE68301B70047EB8FE3B93CB409BD2322BD7]
[DEBUG] gpg.Encrypt: /usr/bin/gpg [/usr/bin/gpg --quiet --yes --compress-algo=none --no-encrypt-to --no-auto-check-trustdb --encrypt --trust-model=always --recipient B5B4BE68301B70047EB8FE3B93CB409BD2322BD7 --recipient D19ACC69CCC4E1726D17034B19115B61E8F25EBA --recipient 0x19115B61E8F25EBA]
gpg: B5B4BE68301B70047EB8FE3B93CB409BD2322BD7: skipped: No public key
gpg: [stdin]: encryption failed: No public key
[DEBUG] Failed encrypt secret: exit status 2
2019/01/21 18:31:59 Failed to encrypt

I seems like the command is receiving something from STDIN which changes the behavious of the gopass binary cat and instead of printing the decrypted and decored secret to STDOUT, gopass is trying to create new secret from the input data but as the public key is remaining, gpg sucks.

When I disable the STDIN for the gopass command, as follow, the command runs correctly and I get the expected output:

gopass binary cat test-gopass/foo/bar < /dev/null

Steps To Reproduce
I use the following Jenkins Pipeline with the sh groovy directive to execute the gopass binary cat command:
node ('jenkins-slave-gopass') {
  stage('Gopass Crypt Unlock') {
    withCredentials([file(credentialsId: 'jenkins-gpg-keys', variable: 'JENKINS_GPG_KEYS_FILE')]) {
      sh 'gpg --batch --import ${JENKINS_GPG_KEYS_FILE}'
    }
    sh '''
      echo "[INFO] Setting up global Git configuration"
      git config --global user.name jenkins
      git config --global user.email<EMAIL_ADDRESS>      git config --global --list
    '''

    dir('/home/jenkins/test-gopass'){
      git credentialsId: 'jenkins-git-credentials', branch: 'master', url: 'https://foo.bar.baz/quax/test-gopass'
    }

    sh '''
      echo "[INFO] Init Gopass secrets store"
      gopass config autosync false
      gopass --yes init<EMAIL_ADDRESS>      gopass --yes mounts add test-gopass /home/jenkins/test-gopass
      gopass binary cat test-gopass/serviceaccounts/tokens/builder
    '''
  }
}

Expected behavior
gopass binary cat test-gopass/foo/bar should print the secret value and not updating it.
Environment

system / container image: Linux/RHEL 7.4
gopass version: 1.8.3 (installed from a binary from github)
gpg version: 2.0.22
Jenkins version: 2.82.2

@abessifi What happens if you perform all your steps in a single sh step instead of spread across multiple steps within the stage?

This seems to be a problem not related directly to Gopass but maybe to Jenkins and how it runs the shell commands.


Water bug fix for #127. Yes it's 2 lines long

Because we directly override water.json within the mod, we did not add ''minecraft:water'' & ''minecraft:flowing_water'' to the entries.

Misclick.

We do not directly override water.json. Refer to line #2 of the JSON, which when set to false specifically does not replace existing tags.

I may have misunderstood exactly what the water.json file does with changing of tags but without the two mc water tags being present, basic fluid physics in water.
This screenshot shows that the water blocks are actually missing the minecraft:water tag with SpookyTime installed (when using current master branch). In vanilla the water blocks do have the tag with them:
http://prntscr.com/ph6iq6

I still question why this only seems to occur for you though, this has to be some issue elsewhere and I'd prefer to fix that issue rather than provide this odd fix.

That being said if we can't find out what is actually causing this, I'm fine with having to do something like this.

And I don't mean this is an issue that you've caused, there just has to be some underlying cause behind this in general.

After a bit of testing using an actual non-dev client the issue is related to eclipse not acting right with resource loaders. I have no idea how to fix this dev side issue. Other than the 2 line hack.

Possibly related to FabricMC/fabric#66.

Closed. Issue was Eclipse, after reinstalling OS and moving to IJ it is no longer present.


✨ Add support for testing with deployment on K3D

Summary
The following steps will deploy/use K3D by setting env-var "USE_K3D".  e.g. USE_K3D=1, or yes, or anything:
setup-kubestellar.sh, run-test.sh, check_pre_req.sh, cleanup.sh, Makefile, ginkgo_suite_test.go
Tested on Ubuntu 22.04.
Related issue(s)
Fixes #

Looks good @eaepstein but the pr-test-e2e workflow fail because it tries to access the old kind-kubeflex context.

@eaepstein few general comments:

Not sure we agreed we should run all our e2e tests on k3d and not just the main 2 examples (multi cluster & singleton)
Maybe it would be better to try and separate the break the setup script and create a different platform dependent one for each platform. In the future we may want to support several other platforms and I suspect we don't want to go into a complex if/else statement  all over our testing scripts
Maybe we can have some generic format that a platform setup script need to comply with, this way we can have our flow agnostic to the platform itself e.g.,

createCluster
deleteCluster
renameCluster
installKubeFlex
etc...

/ok-to-test

Dropping the idea of idea of making the basic test script work with k3d in addition to kind until there is a good reason to do so.


How to use Platform.__setOS()?

When using components that has several implementations for different platforms (for example with .android.js or .web.js extensions), ReactNative uses Platform.OS to identify the current platform. react-native-mock-render is using by default ios, so my tests are failing. It adds a Platform.__setOS() private method to select the platform to be emulated, but I'm not able to use it on my code and there's no examples about how to do it. Could you be able to give me some advice about that?

Ok, at https://github.com/RealOrangeOne/react-native-mock/issues/17#issuecomment-234166764 provides a little bit more information about this topic.

Hi @piranna, Platform.__setOS() was added before we forked from https://github.com/RealOrangeOne/react-native-mock and has not been updated since.
Unfortunately, I'm not familiar with its use and we don't use that particular helper method at Root (we use dependency injection to simulate different operating systems in our tests instead).
If you would like more help with it, you might be better off asking at https://github.com/RealOrangeOne/react-native-mock. Alternatively, if the feature is broken and you need it to work, you could open a PR and we could get it merged and published for you.

Feature is not broken, or at least don't seems so by reading source code. I'm going to continue improving my other tests before digging on this issue.

Closing this as stale.


Categories - Modified

I'm creating a new pull request for categories. I was helping make changes when the other pull request was merged into master. This pull request contains Sushmita's and my code to be reviewed by Darryl.

Sorry... I forgot to switch again. I was trying to fix an error and left the pull request for the original repo.


Dependence on esmf broken in master

Trying:
conda env create -f environment.yml
in the master branch:
ResolvePackageNotFound:

esmf ESMF_6_3_0rp1_ESMP_01 np111py27_2

Tried to change to:

esmf ESMF_6_3_0rp1_ESMP
But then getting the hdf5 1.8.* troubles.

See also issue #72 and #74

Also cdo seems to be broken since a week or two. I managed to create an env file with cmor-3.2.8 and cdo-1.8.2 which seems to work, I just have to fix some tests before I push it.

Can you check again with the newest master branch?

I have still troubles, I get now the following:
Trying:
conda env create -f environment.yml
in the master branch:
ResolvePackageNotFound:

intel-openmp 2018.0.0 hc7b2577_8

Can you check once more with the newest master branch?

Ok for linux works fine now, so close this issue.
However for Mac still unresolved packages.


Fix most lints

Overview
This PR picks off all the "minor" linting warnings after merging #52. That means anything that was purely stylistic or fixable via eslint --fix.
There are about 50 remaining warnings, mostly from unicorn/filename-case and various @typescript-eslint rules that are going to be more involved to fix.
Test plan
Due to number of files touched, IMO this PR requires smoke testing with actual hardware. The riskiest move here is probably the import reordering, since it could cause changes in behavior if we have any problematic circular dependencies.

Superseded by #70


Cannot build on node 12

Sleep version 3 does not compile with node 12. Is it possible to update it to a newer version?

Thanks for your report, and @korczis for the patch. I've published a new version with the bumped dependency.


Why last-update 	 2023-02-23T02:00:34+0000 ?  Adjust update strategy

I highly doubt that there were no changes to any of the repositories since then.  Don't we run some cron job which tests if there are updates to already known datasets? obviously we might be not running enough of updates but we should get some and we should do them by looking first at the least recently updated -- i.e. with the oldest info_ts, and thus result in them being updated even if there is no actual updates from the remote.

There is a cron job running, but I don't think the update logic is right as mentioned in https://github.com/datalad/datalad-registry/issues/136. I will put in some changes to make update working starting with re-organizing the local cache as discussed in https://github.com/datalad/datalad-registry/issues/125.

PR #238 closes this issue as well.

if it does -- add Closes #238 in the description of that PR and the issue would be closed when that PR is merged


Automatically Generate Configuration from Source Files

Thanks for rustfmt! It's awesome!
As rustfmt has been growing, it's gained quite a few configuration options. This is great because having more options makes it possible for anyone to adapt rustfmt to their specific needs.
Unfortunately, having lots of options can also add a very large barrier to entry for new users of rustfmt.  Right now, users can either just take the default configuration and live with it, or go through the process of tweaking the settings until they figure out what to do.
The idea I have to address this is from a related tool for JavaScript/ES6 called ESLint. It has a really cool feature where if you just type eslint --init, you can run a program that will automatically determine which configuration best suits your code. The process has its hiccups, but is pretty seamless most of the time. Here's an article about the exact process in detail.

It would be great if we could implement a similar kind of thing for rustfmt. Someone could type rustfmt --init, answer a few questions, and then be completely setup!
I think a feature like this could greatly improve the usability and adoption of a tool like rustfmt. Flexibility is great, and it's even better when you don't have to do very much to take advantage of it. :smile:

@topecongiro Could this issue be reopened? I still think it would be very useful to have a way to automatically determine a rustfmt configuration from a user's source code. It looks like the PR was closed due to inactivity, so I don't see why this issue should be closed as well.


Erasing T for isErr

The function isErr is not erasing the T of Result<T, E> even though it could do so, and its partner function isOk is already erasing E.
Changing the signature to export function isErr<T, E>(val: Result<T, E>): val is ResErr<never, E> would allow writing if (isErr(value)) { return value; } analogous to if (isOk(value)) { return value; } which already is possible.

Hi @maxmuen, thanks for raising this, it should be a fairly trivial change - would you mind creating a PR?


Add (U)?Int(128|256) integer types.

I had to implement read_items/write_items given that this size of integers is not supported by Python's struct.
Not sure it's ideal to have these new types transitively inherit from FormatColumn, but I wanted to keep the check_items/before_write_items, and this works fine.

Thanks! Please add unittests about this.

Done.

@long2ice when it will be merged?

#57


What are they use of args "rbeg" and "rend" of sanityinc/isearch-exit-other-end?

https://github.com/purcell/emacs.d/blob/add16e79c067a9de36ff74909691f44f8b18efde/lisp/init-isearch.el#L36
Thanks.

Looks like they're unnecessary. :-)

Thanks.
I can't think of why the original author would do this.
Maybe because this is actually part of the original author configuration.


Page titles include "-" with no text after

The title of the page appears incomplete, see below where title of a page is "Standards -"
All pages have a trailing " - "
I think ours should be something like:

< page title > - Engineering Guidance and Standards

or

< page title > - Home Office Engineering Guidance and Standards

The home page itself should only have the latter part of the title.

See the below sites for examples of this:
https://design-system.service.gov.uk/patterns/addresses/
https://design.homeoffice.gov.uk/patterns/access-a-service

It appears these configurations can be controlled using the xgov-11ty plugin header options, namely organisationName and productName.
This results in the text for both of these appearing in the title and also the header of the pages. Formatting of this may need to be considered along with the use of the existing logo which contains text within it.

Yes, we had issues attempting to resolve this when we were first configuring the site and what you see at the moment was the minimal acceptable workaround.
An option might be to raise a PR upstream on https://github.com/x-govuk/govuk-eleventy-plugin/blob/main/layouts/base.njk, to add condition on the inclusion of the organisationName as they have done with productName
{% block pageTitle %}   {{- title if title -}}   {{- " (page " + pageNumber + " of " + pageCount + ")" if pageCount > 1 -}}   {{- " - " + options.header.productName if options.header.productName -}}   {{- " - " + options.header.organisationName -}} {% endblock %}


Add macos-14 arm64 runners

This PR adds macOS arm64 runners for #1672
The only issue I'm facing is python 3.8 and 3.9 aren't working with the current setup-python action.
https://github.com/actions/setup-python/issues/696
Wheels can still be made, python 3.8 and 3.9 are supported with the cibuildwheel action.
python-3.9 is the version of python included on macOS, so its annoying the test aren't running.
As for python-3.7, it was never supported for apple silicon as far as I understand.

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

Project coverage is 84.11%. Comparing base (2b9c3a4) to head (7e687fe).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1739      +/-   ##
==========================================
+ Coverage   79.95%   84.11%   +4.16%
==========================================
  Files         197      198       +1
  Lines       21879    22241     +362
  Branches     4342     4689     +347
==========================================
+ Hits        17494    18709    +1215
- Misses       2252     2610     +358
+ Partials     2133      922    -1211

Flag
Coverage Δ

py-unittests
84.11% <ø> (+4.16%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 107 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

Hi @JeanChristopheMorinPerso @markreidvfx  The CI passed on this PR, so I merged it. However, the runners "in real life" are failing now that it's merged. Any idea what's going on?

looks like a timeout of some kind with ssl. Retrying the job I think might fix it
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/ssl.py", line 1042, in _create
      self.do_handshake()
    File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/ssl.py", line 1320, in do_handshake
      self._sslobj.do_handshake()
  TimeoutError: [Errno 60] Operation timed out


s4742562 - Siamese Network

Implemented a Siamese Network to classify ISIC2020 dataset images as 'benign' or 'malignant'.
Built a margin-based classifier for final classification based on learned embeddings.
Trained the Siamese Network using contrastive loss to ensure meaningful embedding distances.
Results available in README

This is an initial inspection, no action is required at this point
Recognition Problem : total : 19
Solves problem: The solution is well-suited to the problem (4).
Implementation functions as intended (2)
Good design: Well-designed (1)
Commenting: Clear and sufficient comments throughout the code. (1)
Difficulty: Hard (10)
Note:

The model performs well for benign cases and struggles with malignant.
According to the reported confusion matrix, Accuracy is 90.6 and Specificity is 91% but Sensitivity is 67.5%.
tSNE training Shows distinct clusters, indicating some separation between classes in the training data. however,  tSNE validation displays less separation, with more overlap between clusters, suggesting reduced model generalization on validation data.
- Also, the validation set has nearly the same number of points as the training set in the t-SNE plots, which is unusual
please ensure the training and validation sets are correctly split, make sure there’s no overlap between training and validation samples, and avoid oversampling in validation.
Using an algorithmic approach to undersample the majority class or oversample the minority class is indeed recommended over random sampling, especially in the context of medical imaging to maintain informative samples, improve generalization, and avoid loss of critical information.

Hi @hanemma7moud ,
Could you clarify where these marks were lost? Regarding the feedback on the t-SNE plots, I have to point out that assuming similarity in point count just by looking at a 2D graph doesn't seem reasonable. These plots are, after all, only showing two feature values, so naturally, overlapping will occur with similar points. Adding jitter would just clutter the visual, and shifting to a 3D format or a heatmap doesn't add meaningful insight here.
On the note about the train-test split, I’m confused by the suggestion it wasn’t correctly handled, especially since I clearly provided the actual class counts for each split in my README.md. Just to be thorough, here’s that table again for reference:

Dataset Split
Size
Class 0 Count
Class 1 Count

Train
23,186
11,593
11,593

Validation
3,313
3,255
58

Test
6,626
6,509
117

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
No Header blocks. -1
Recognition Problem
Good solution to problem.
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found. Has data splits.
Difficulty : Hard.  Siamese network (Hard Difficulty)
Commit Log
Some/Adequate Meaningful commit messages. -1
Some/Adequate Progressive commits. -1
Documentation
Readme :Good.
Model/technical explanation :Good.
Description and Comments :Acceptable. References missing.-1
Markdown used and PDF submitted. PDF Checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). On time.
No Feedback required.
Request Description is adequate. -1
TOTAL-5

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Cant merge because of conflicting changes to main repo files (gitignore). Please update for merge, doesn't affect grade.


Version 1.4.2

Hi Jeff
Thanks for the awesome gem.
I have added some functionality to it to resemble ActiveRecord model even better:
1- Where like querying: Model.where("id > 3 AND value = 4")
2- Before_filter and after_filter
3- Support for nested models.
I have wrote new test for all the new functionality and run all the tests against all the ruby versions and specs, I just had to specify rake version 10 in the travis.yml file because rake version 11 is not compatible with ruby <= 1.8
rake build, rake install are done too.
I wanted to push it to rubygems.org but I don't have access, my user name: rudyyazdi
Please let me know if there is anything else I have to do.
Thanks and Cheers,
@zilkey

@rudyyazdi - I'm interested to hear your use cases for the filters.  When we built this library we imagined it being quick config data that didn't really "deserve" a database table for whatever reason.  I've mostly seen it used to provide hard-coded default data, in which case you fully control the data, and there's no need to validate.
What are you doing that requires filters here?

@zilkey thanks for the comment, I had the case that I had to import a bunch of objects from file, (products, users, purchases and transactions) and then purchases and transactions would go on after import for a while. I modeled the system with those objects talking to each other (rails like models) but I had to catch the anomalies before the objects were created :)
There are probably better ways to do it, but this model worked great for me as it would fit seamlessly in case we wanted to add a database in the future :)
If you think that's unnecessary please let me know and I can remove the filters altogether.
Cheers,

@rudyyazdi Hello, sorry for late reply 😓
According to https://github.com/zilkey/active_hash/pull/121#issuecomment-219635399, 2 and 3 are sadly, but unnecessary.
But I think

1- Where like querying: Model.where("id > 3 AND value = 4")

seems to useful for some use cases.
If you are still interested in this feature, please separate your patch by the feature and create as another PR.
Thanks.


Freshdesk connector throwing error with correct credentials

Even with correct auth, the Freshdesk connector is producing the following error:
subprocess.CalledProcessError: Command '['python3', '/usr/local/lib/python3.10/site-packages/mage_integrations/sources/freshdesk/__init__.py', '--config_json', '{"api_key": "******************", "domain": "*********", "start_date": "**********"}', '--log_to_stdout', '1', '--settings', '/home/src/mage_open_infra/pipelines/quiet_wood/data_integration_catalog.json', '--state', '/home/src/mage_data/mage_open_infra/pipelines/quiet_wood/freshdesk/tickets/tickets_test_state.json', '--query_json', '{"_end_date": null, "_execution_date": "2023-08-08T10:39:35.430198+00:00", "_execution_partition": "30/20230808T103931", "_start_date": null, "_offset": 0}', '--selected_streams_json', '["tickets"]']' returned non-zero exit status 1.

https://mageai.slack.com/archives/C03HTTWFEKE/p1691492121462389

An update: It seems there is some difficulty in replicating these errors, but I am still getting this error for tickets. I also get different errors for other tables/endpoints. Two tables load (Agents and groups), however, so I don't think it is my connection that is the issue. I can post other errors here, but I have posted one other in slack.

Hi @PSmythOI , do you still have this issue?

@wangxiaoyou1993  I can probably go back and test this to see - my company has since moved on from Freshdesk


macOS and Linux projects will re-import as Windows

Release Type: Official Release
Version: <IP_ADDRESS>9-beta
Platform(s): Mac, Linux
Describe the bug
Upon reload, Stride Game Studio will identify macOS and Linux projects as Windows projects. This may not result in any adverse effects beyond Stride Game Studio.
To Reproduce
Steps to reproduce the behavior:

Create a "New game" project
Select Windows, Mac, and Linux as the Platforms
When Stride Game Studio has loaded, note the platform icons next to the Mac and Linux projects
Press the "Reload current project" button
When the project has reloaded, note the updated platform icons

Expected behavior
Mac and Linux projects should retain their respective platform icons upon reload.
Additional context
This issue cannot be replicated in Stride <IP_ADDRESS>0

The error still exists, but only for the macOS platform. Project platform is determined by StridePlatform msbuild variable. If you set the variable in the csproj, though, it will work correctly:
MyGame.macOS.csproj:
<PropertyGroup>
...
    <StridePlatform>macOS</StridePlatform>
...
</PropertyGroup>

VSProjectHelper.LoadProject(path) <- after loading the project from this method we get StridePlatform eq "Windows" for macOS and Windows platform and "Linux" value for Linux project (as it should be), I coudn't find why

Hi @Jklawreszuk, nice find! I'm able to re-replicate this bug in <IP_ADDRESS>8 but my csproj is missing  for all 3 platforms (however, adding it to the macOS.csproj fixes the issue).


View scaling without default font (segoe) is broken

Version of Archi
5.3.0
Archi Plug-ins
N/A
Operating System
Windows 11
Expected Behaviour
Zooming the view page, or exporting the view at greater than 100% with an alternative font should render with the correct font at the correct size.
Actual Behaviour
Text Boxes are cut off, font is not rendered correctly.
In these examples, the "Business Process" has a font of Segoe, the others are "Roboto Regular"
100% -
150% -

Steps to Reproduce the Behaviour

Change to alternative font (Roboto downloaded from Google)
Zoom View or Export at > 100%
Render is broken

Hi, I tried to reproduce this using Roboto Regular but it was OK for me. Your Roboto font looks different than mine. I'm using the one in C:\Windows\Fonts\ Could you try a different version of Roboto? Having said that, the Eclipse Draw2d framework does have some difficulty rendering some fonts, so you'll have to avoid these.

Just tried it with a different version of Roboto from dafont.com and this time it's even wierder...

Looks almost like the issue in 974
I'll see if there is another version of Roboto out there...

Working now!

Looks like my fonts were installed as a Local user...
Removed them from  %LOCALAPPDATA%\Microsoft\Windows\Fonts\ , installed the fonts for all users to Windows/Fonts, rebooted, and it's happy now!
Cheers


UDDS loading profile

Is there support for UDDS loading profile in pybamm. I can see various researchers using this loading profile in batteries in MATLAB.

Look for US06 in this example https://github.com/pybamm-team/PyBaMM/blob/develop/examples/notebooks/change-input-current.ipynb. You can replace our US06 cycle with your UDDS cycle. If you could add the UDDS cycle here that would be really helpful!

@tinosulzer Thank you for your response. I will check out the example. Sorry I don't have the UDDS cycle profile with me. Hope somebody else contributes to that.

You will need to provide the UDDS cycle to be able to simulate it

Yeah I got it :+1:


fixed bug where beacon newsletter sent was impoperly updating

fixes #461

So long as we test in staging first LGTM :+1: :ok: :microscope:


irmin-bench: Add tree.exe's compilation to runtest

Same kind of mistake as #1286. Won't happen again!

The CI seems unhappy because this requires some new test dependencies in irmin-bench.opam.

irmin-bench.opam: changes needed:
  "bentov" {with-test}                     [from bench/irmin-pack] (missing {with-test} annotation)
  "mtime" {with-test}                      [from bench/irmin-pack] (missing {with-test} annotation)

I've just added the 2 annotations, but is it the right way to proceed? progress doesn't require that annotation because it is used from bench_common, that's unsatisfying to treat them differently.


Cannot disable consul health indicator in Grails application

Follow up issue related to #234.
I have management.health.consul.enabled = false set in my configuration, but the consul health indicator is still present in my health check.
@spencergibb I realize now that this problem only happens when I include spring-cloud-starter-consul-all in a Grails application.  It doesn't happen in a plain Spring application.  Do you have any intuition about this?
To make the issue occur, I used grails create-app to generate a new Grails application, set endpoints.enabled: true in the configuration file (to make the health endpoint available), then included org.springframework.cloud:spring-cloud-starter-consul-all with gradle.  That caused the health endpoint to appear and to show the Consul health indicator, as well as 3 others: "diskSpace", "hystrix", and "refresh".
By putting management.health.***.enabled = false in the config file, where *** is any of "diskspace", "hystrix", or "refresh", I can turn the respective indicator off.  But, for some reason, this does not work for the Consul health indicator.
Here is a link to a project that recreates the problem:
https://github.com/theAmneziac/grails-consul-health-indicator-example
Thanks for your time.  I realize the twist of this only occurring in Grails applications is annoying.  If you don't have an intuition about it, or you think it's an issue with Grails and not with Spring Cloud Consul, I totally understand.

Can you try importing the spring-cloud-dependencies bom rather than the consul dependencies?

We experienced the same problem as well.
The issue is that all the consul stuff are initialized in the very beginning before all your configs are loaded. So what I did is to put the management.health.consul.enabled = false in the bootstrap.yml and this would disable the creation of ConsulHealthIndicator.

The issue is that all the consul stuff are initialized in the very beginning before all your configs are loaded.

Who does "your"  refer to?

@chengchen That worked perfectly, thank you for posting.
@spencergibb sorry I didn't get back to you about your bom idea, I was going to try it later when I was less busy.

@spencergibb Sorry about the misunderstanding, I was talking about the normal application.yml
When ConsulAutoConfiguration decides if it needs to add the ConsulHealthIndicator, he will check if management.health.consul.enabled = true, or management.health.defaults.enabled = true OR it's not configured. Since consul are initialized before almost everything, the application.yml is not yet loaded (ConsulAutoConfiguration thinks management.health.defaults.enabled is not configured), it will simply creates the ConsulHealthIndicator.
However, if you add the bootstrap.yml, then the configuration will be taken into account.


Upload to PyPI

Are there any plans to package those? It is kind of hard to setup dependencies for HTML TOC extension,

@lepture pip fails to install mistune-contrib now. This is the log from Heroku https://github.com/gratipay/inside.gratipay.com/issues/410#issuecomment-158487528

@techtonik Ok. Published to PyPI.

@lepture thanks! You saved my day. =)
I was going to create a separate "virtual machine" for Heroku just to get this working. :+1:


Documentation

I am able to set up the environment locally. Can anyone guide me how to and where to write code to send messages using playwright ?

It would really help to get me started also, if I can have some wiki's or brief documentation to use..
Right now unable to figure out how to start it..
npm run launch:local  did help me start the chromium but than after i couldn't figure out where to keep the custom code. From what file or which place can I call the WPP functions to send messages and all.
I tried to modify the tool/launchLocal.ts but unable to use WPP object over there..
It would be a great help if some can share a small snippet to simply send message from local environment..
Thanks

There are a short example in the README file: https://github.com/wppconnect-team/wa-js#playwright
npm install playwright-chromium@latest @wppconnect/wa-js@latest
import * as playwright from 'playwright-chromium';

async function start() {
  const browser = await playwright.chromium.launch();
  const page = browser.newPage();

  await page.goto('https://web.whatsapp.com/');

  await page.addScriptTag({
    path: require.resolve('@wppconnect/wa-js'),
  });

  // Wait WA-JS load
  await page.waitForFunction(() => window.WPP?.isReady);

  // Evaluating code: See https://playwright.dev/docs/evaluating/
  const isAuthenticated: string = await page.evaluate(() =>
    WPP.auth.isAuthenticated()
  );

  // Sending message: See https://playwright.dev/docs/evaluating/
  const sendResult: string = await page.evaluate(
    (to, message) => WPP.chat.sendTextMessage(to, message),
    to,
    message
  );
}

start();


Active Challenge Page

User Story
As a user
I want an overview of my active challenge,
so that I can see the details and my progress.
Description

Acceptance Criteria

[ ] there should be an image and the title of the challenge
[ ] there should be an info box with the challenge data
[x] only the activities for this challenge should be shown

Tasks

[ ] create a new branch
[ ] add image
[x] add title
[ ] create component InfoBox

[ ] shows challenge distance, completed distance, completed in %, total duration

[ ] check if a challenge is active

[x] false -> text: you have no active challenge and button to challenges page

[ ] true -> shows challenge specific data

Complexity
M

well done, @UlrikeWerner  👍


Call the SSL callbacks not the CTX ones

Use the callbacks associated with the SSL object instead of the SSL_CTX object in functions dealing with the SSL object rather than the context.
Fixes #2538.

Considering the original age of #2538, I imagine this applies to 1.0.2 and 1.1.0 as well.

Merged (after a slight edit of the commit message).
master: d61461a7525322d188f9c6e3f90cfc93916cc636
1.1.0: e7dbb4edce65446b8cd69b86ca73a48a1d7ea02a
1.0.2: 5247c0388610bfdcc8f44b777d75ab681120753d

Hello good people. I think that this patch may be causing problems with wpa_supplicant: https://bugs.archlinux.org/task/54233 . I'm not sure if I read it right, as I didn't read the code thoroughly, but wpa_supplicant is setting the password in ssl_ctx:
SSL_CTX_set_default_passwd_cb_userdata(ssl_ctx, passwd);
if (
    SSL_CTX_use_PrivateKey_file(ssl_ctx, private_key,
                                SSL_FILETYPE_ASN1) != 1 &&
    SSL_CTX_use_PrivateKey_file(ssl_ctx, private_key,
                                SSL_FILETYPE_PEM) != 1 &&
    tls_read_pkcs12(data, NULL, private_key, passwd)) {

This is more a question than statement - may it be an issue? Should it be fixed in wpa_supplicant or maybe the problem is on the openssl side?


修复gxml字符集转换的并发安全问题

修复并发安全问题,改为如果非UTF8字符集则先做字符集转换

Codecov Report

Merging #59 into master will increase coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #59     +/-   ##
=========================================
+ Coverage   44.57%   44.68%   +0.1%
=========================================
  Files         154      154
  Lines       13655    13655
=========================================
+ Hits         6087     6102     +15
+ Misses       7008     6996     -12
+ Partials      560      557      -3

Impacted Files
Coverage Δ

g/container/garray/garray_normal_interface.go
77.01% <0%> (+1.19%)
:arrow_up:

g/container/garray/garray_normal_string.go
72.2% <0%> (+1.2%)
:arrow_up:

g/os/gcfg/gcfg.go
48.19% <0%> (+1.35%)
:arrow_up:

g/util/grand/grand.go
95.55% <0%> (+2.22%)
:arrow_up:

g/os/gtimer/gtimer_entry.go
89.53% <0%> (+3.48%)
:arrow_up:

Continue to review full report at Codecov.

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


docs: add a link for the Druid SQL tutorial

Description
This PR add a link for the Druid SQL tutorial that uses Jupyter to the tutorial-jupyter-index.md file.
This PR can be merged after PR 13465, PR 13342, and PR 13588.
This PR has:

[ x  ] been self-reviewed.

@ektravel , do you want to remove the lines referring to Segment sizing and partitioning so we can merge this PR? Thanks!

@techdocsmith I've removed the lines referring to Segment sizing and partitioning. Thank you for fixing the merge conflict.


Expose reconcile status as metrics to allow monitoring

Describe the problem/challenge you have
One thing we need is at least be able to expose the reconcile status to an external monitoring sysytem. We have seen a cluster that was running fine for several days until we realized that it was actually failing to reconcile due to someone mistakenly changed the service account role.
This cannot be identified until somebody executes kubectl get apps
Describe the solution you'd like
Something like kubernetes_apps_reconcile_status metrics being exposed where 0 = success , 1 = fail , 2 = unknown then i think it's easy to monitor

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Hi,
I am currently trying to develop a pull request. Will try to submit in few days for review.

PR was merged and shipped some time ago. thank you again @mhoshi-vm


Kernel crash on remote server

Thanks for creating a wonderful notebook.
After installing and cloning the repo on a remote server (following your instructions), I tried running the notebook. But I find that the kernel crashes at the cell where I load the QM9 dataset.
Any idea why this may be the case? Perhaps because it's too huge for my server to handle or am I missing something else?
Regards,
Madhav

Thanks, Chaitanya.
Yes, I was referring to the 101 notebook.
I tried cloning and running on a remote server I have access to, mainly because Colab times me out after a while and the pytorch package installation cell takes ~42 mins to finish running on Colab.
Anyway, I shall take a closer look.

You will have to install the library versions stated in the 101 collab notebook in order to run that; it may be different package versions compared to the full dojo repository.
Closing the issue, feel free to re-open or ask further queries as new issues.


REST client tests

Closing: https://www.atlassian.com/migration/assess/journey-to-cloud

Support for Server products ends on Feb. 15, 2024


'wait_until' method with 'text:' option doesn't work correctly without specifying timeout value

Environment

Operating System: Windows
What Gems / Versions you are using: site_prism 2.14, Capybara 2.18.0
How you're interacting with the Framework: Cucumber Tests
Browser/s Driver/s being used: Chrome 68.0.3440.75, ChromeDriver v.2.40, Selenium 3.11.0

Expected Behavior
'wait_until_<element_or_section_name>_visible' method works with 'text:' option without specifying timeout value as described in documentation here:
https://github.com/natritmeyer/site_prism#methods-supporting-capybara-options
@results_page.wait_until_<element_or_section_name>_visible(text: 'Some ajaxy text appears!')
Actual Behavior
Method ingores text: param (as I think) until I set timeout param explicitly
Steps to reproduce
Capybara config:
Capybara.configure do |c|
  c.default_selector = :xpath
  c.default_max_wait_time = 30
  c.ignore_hidden_elements = true
end

Siteprism config:
SitePrism.configure do |c|
  c.raise_on_wait_fors = true
end

Page class example:
class Github < SitePrism::Page
  set_url 'https://github.com/'

  element :login_link, '//a[@href="/login"]'
end

Cucumber step example:
When(/^I look at login button$/) do
  @github.load
  @github.wait_until_login_link_visible(text: 'any text')
end

When I use only text: option then step finished succesfully
If I set timeout:
@github.wait_until_login_link_visible(10, text: 'any text')
then step is failed on this method as expected

Removed options and issue still reproducible.
Attached for_site_prism.zip

Cheers. Will triage ASAP

Ok so the short answer, is because that portion of the README was written in This commit https://github.com/natritmeyer/site_prism/commit/3ba7944a104804f825fc9e1f48d4bda785a0a7a9 (Which is ancient), I don't believe the way the API was designed will facilitate what you need.
The quick fix is to change your code to @github.wait_until_login_link_visible(nil, text: 'any text') -> Note that this is actually mentioned in some code comments in the README above for the wait_for methods.
I think in essence you've spotted a slight flaw (And in internal test documents, we cover this), where our API design sets the timeout separately to every other runtime argument. And in essence it shouldn't
I've added this to the roadmap for v3.0 as this needs fixing, but this will break every single metaprogrammed method from going from.
wait_for_element(10) to wait_for_element(wait: 10)
With the latter being more desirable due to being hashable and passable as a query.
Once we've made the minor (But breaking change), mentioned. You will be able to do your code correctly.
Now thats the good news......
The bad news....
We need to get a minimum of 4 more code releases out of features / tweaks I want to make, as well as get some more of the backlog of tasks completed.
I also need to chat to the original author as this would be a reasonably large change to the docs and methodology.
So an ETA for the changes to make your code work as you expect is a long way off sadly.
TL;DR - add nil as your first parameter. Or you can use the has_? methods which already work in this way (And facilitate waits via implicit waiting)

@nbulavin - This is fixed in the current master branch and will be cut as an alpha 3.0 release (Alongside the other items in master), sometime in the coming weeks (Probably around 25th-30th)


Animation on phases

Just for enhancement, I would recomend a screen animation between investigators and mythos phase.

Also add an option to disable it.

I have worked on this, and I have something working.
The result is quite good and add to the ambiance.
The 2 screens are fixed (no animation), and last for 3 seconds.
I don't think we really need a skip button, as 3 seconds is a very short time but feel free to try it out and tell me :)
Please note that there are no additional actions required by the players, the screens just come and go at the right time.
See screenshots below, feel free to comment on this, and ask for improvement.:

Commit was done in my branch  mom_ui_update_clean, which already has a pull request.
I can't really start it from a new branch, as I'm working one files that have taken many other changes for ui improvement.

Hi @FASTNAVi , ok,  I will do that no problem.

Beta test version available with implementation of this issue:
https://github.com/BenRQ/valkyrie/releases/tag/release%2F2.1%2FBrynhildr_2.1.0b
Please close this issue, if you are happy with this fix.


[FEAT] Rename useSelect's queryResult to query.

Is your feature request related to a problem? Please describe.
queryResult name is long.
const { queryResult, defaultValueQueryResult } = useSelect()

Describe the thing to improve
We can delete the result part for the name. this way it will be more readable.
const { query, defaultQuery } = useSelect()

Additional context
The following hooks should be refactored too.

useCheckboxGroup
useRadioGroup
useAutocomplete

I'm not sure about this as a change - query and queryResult are semantically different. I would expect query to refer to  what is being requested (like SQL queries).
If you wanted to shorten it, I think result works better and keeps the same meaning


Cannot read property 'props' of null migrating to webpack 2

Hi, I have migrated to webpack 2 and the project is breaking for that:
Cannot read property 'props' of null
TypeError: Cannot read property 'props' of null
    at RadioButtonGroup.hasCheckAttribute (/Users/albert.olive/Repos/web/node_modules/material-ui/RadioButton/RadioButtonGroup.js:122:25)
    at RadioButtonGroup.<anonymous> (/Users/albert.olive/Repos/web/node_modules/material-ui/RadioButton/RadioButtonGroup.js:102:20)
    at forEachSingleChild (/Users/albert.olive/Repos/web/node_modules/react/lib/ReactChildren.js:51:8)
    at traverseAllChildrenImpl (/Users/albert.olive/Repos/web/node_modules/react/lib/traverseAllChildren.js:77:5)
    at traverseAllChildrenImpl (/Users/albert.olive/Repos/web/node_modules/react/lib/traverseAllChildren.js:93:23)
    at traverseAllChildren (/Users/albert.olive/Repos/web/node_modules/react/lib/traverseAllChildren.js:172:10)
    at Object.forEachChildren [as forEach] (/Users/albert.olive/Repos/web/node_modules/react/lib/ReactChildren.js:71:3)
    at RadioButtonGroup.componentWillMount (/Users/albert.olive/Repos/web/node_modules/material-ui/RadioButton/RadioButtonGroup.js:101:32)
    at /Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactCompositeComponent.js:348:23
    at measureLifeCyclePerf (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactCompositeComponent.js:75:12)
    at ReactCompositeComponentWrapper.performInitialMount (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactCompositeComponent.js:347:9)
    at ReactCompositeComponentWrapper.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactCompositeComponent.js:258:21)
    at Object.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactReconciler.js:46:35)
    at ReactDOMComponent.mountChildren (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactMultiChild.js:238:44)
    at ReactDOMComponent._createContentMarkup (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactDOMComponent.js:653:32)
    at ReactDOMComponent.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactDOMComponent.js:520:29)
    at Object.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactReconciler.js:46:35)
    at ReactDOMComponent.mountChildren (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactMultiChild.js:238:44)
    at ReactDOMComponent._createContentMarkup (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactDOMComponent.js:653:32)
    at ReactDOMComponent.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactDOMComponent.js:520:29)
    at Object.mountComponent (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactReconciler.js:46:35)
    at ReactDOMComponent.mountChildren (/Users/albert.olive/Repos/web/node_modules/react-dom/lib/ReactMultiChild.js:238:44)

I'm using material-ui v0.18.1.
Any ideas?
Thanks!

I guess you have a wrong import.

That's weird,
If I add this line:
export default injectIntl(CSSModules(Filters, styles, { allowMultiple: true }));
instead of:
export default injectIntl(Filters);
works.
I'm trying to find what is happening here but with this error stack trace it's difficult to find.

Sounds like it's not an issue with Material-UI then, I'm closing. Thanks for the update.