Update properties section of model details and version details

Closes: RHOAIENG-6996 and RHOAIENG-13057
Description
This PR aims to update Properties section of model details page and version details page:

Table expands when a user clicks on Add property button
New row appears in the properties table with the text input for Key field automatically gets the cursor focus

The description input field of model details and version details pages also get cursor focus.
Labels of model details and version details page are displayed without truncation.
https://github.com/user-attachments/assets/b9a18b8d-af43-4e47-9d1e-cccc177dd421

How Has This Been Tested?

Try to add new row to Properties table with more than 5 property, so it will expand and the row will autofocus.
Make sure the description input field gets auto focus when we click on edit button of description section in model details and version details page.
Make sure the labels are not truncated any more in model details and version details page.

Test Impact
Added and updated cypress tests.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@yih-wang Could you please take a look at the above screenshots?

LGTM, thanks @ppadti !

I didn't find these in this PR while testing:

The list should follow the creation order (oldest on the top, newest at the bottom), not the alphabetical order
The fix for RHOAIENG-13057 - can't see the tooltip with full content for truncated labels
Otherwise lgtm.

we can't sort it by creation order, because we only have key/value and decided to not to do it. Mentioned here in the comment section.
we decided yesterday during standup to show the label without truncation, so no tooltip required. comment mentioned here in the issue.

@ConorOM1 test-id changes in this PR


Add basic API routes

Added new vendored packages to tqweb
Used RegisterAll function to simplify calls
Added routers for currently defined routes
Created data processing route endpoints.

Codecov Report
Attention: Patch coverage is 0% with 72 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (b88c118) to head (b6ce17c).

Files with missing lines
Patch %
Lines

server/route/route.go
0.00%
71 Missing :warning:

server/server.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff          @@
##            main      #7   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          4       4
  Lines         36     105   +69
=====================================
- Misses        36     105   +69

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


Set clusterLevels

Various parameters to set the zoom levels. To set the initial zoom level when map view first
launches, specify zoom value within the <INIT_LEVEL> element.

Zoom level to scale:
double scale = 591657550.5 / pow(2, zoomLevel);

Scale to zoom level:
int myInt = log2(591657550.5 / _agsMapView.mapScale);

Add zoomMap and scaleMap to control the scale of map view.
Remove scale property in centerMap method and initialMapCenter property.


package.json - remove unneeded usage of yarn

@chmln it looks like the rest of the coc plugins and coc-nvim itself relies on yarn. If you look at the install scripts, they even install it and use it. Plus if you use the native coc-install command it utilizes yarn as well to install the plugin. I'm assuming that we'll want to stay consistent and use yarn.

@ckipp01 see https://github.com/neoclide/coc.nvim/wiki/Using-coc-extensions#use-vims-plugin-manager-for-coc-extension.
Plugins can be installed via vim-plug and npm install. However, this particular one fails to install due to relying on yarn in build scripts.

can I suggest using npm-run-all instead, which automatically uses npm or yarn depending on what it was originally invoked with? Also, it will run in Windows, which the && might fail in.


Subaru: universal signal for ACC speed units

Description
This PR updates Subaru Global and Preglobal ACC speed units signals in carstate. Both previously used signals were unreliable and signal values were different across markets/models.
Prerequsite: https://github.com/commaai/opendbc/pull/414
Verification
Both signals have been in use in subaru-community branch since beginning of July and ACC speed has been correct for all tested models

This looks like a nice cleanup, but there may also be a cruise speed signal that doesn't change units. Let's get this merged (left a comment on the DBC PR), but after that, we may be able to find a new speed signal that would make this obsolete.

I updated preglobal to use Dash_State2 UNITS signal to match opendbc PR


[Node] Refactor

This is to discuss how to refactor node.
Currently node when created there is an internal NodeConnection that creates two threads one that listens to incoming messages and one that sends messages to peers.
I suggest we change that to use the AsyncLoop and use the away on incoming and outgoing messages.
Basically making node async.
There are methods in Node that are not used bu the fullnode that can also e removed.
I would also propose to replace the event system of behaviours to us Rx signals.

In this context, I was thinking about MessageReceived subscribers to define which messages they are interested in instead of being called for all messages. Most of the subscribers only want to see one or two types of messages. Currently everyone deals with every message ...

Do Rx signals execute in parallel or in serial fashion?

also what is the advantage of Rx signals, why would you like to use them?

Rx is sequential, no solid preference to Rx, you just get a signalling system already built, I prefer to move away from events.
Yeah would be good to have some sort of subscriber per message type system.

I can't see the reason of using a signalling that is not processing things in parallel. If we have serialized processing and we want it, it might be better to allow things like returning a result of a subscribed processor. This would allow you to, for example, end the propagation of the event to further subscribers if you decided to.
So basically you would do:
foreach (subscribed handler)
{
   should we stop := execute it
   if (should we stop) break;
}


Expand employment profiles

Current, the user profile consists of a few questions, two free-form Markdown areas, and a list of skills. This should be modified to provide the following sections, each editable on their own:
General Information
This is the current information: actively seeking, continent, region, professional bio, remote, full-time, experience, publicly searchable, publicly linkable (new for v3).
Skills
This is the current skill/description combo; list these, and allow users to edit them separately from the general information.
Employment History
This is a new section which can contain one or more entries describing the user's chronological work history. It will consist of:

Employer
Start Date (will display as month/year)
End Date (or "to present")
Title / Position (optional)
Description (optional, Markdown)

These will be sorted descending by end date ("to present" first), and ascending by start date if there are multiple continuing positions.
Finally, update the profile view page to display employment history just above the "experience" general field. Additionally, provide a "print view" of the profile page without any of the application menus, footers, etc. Both these should support public viewing if the user has said that they want their employment profile publicly linkable.

As part of this implementation, three visibility levels were created for employment profiles; Private (logged-on users only), Anonymous (location and skills only for public users), and Public (linkable and shown in public search). Add a fourth - Hidden, which only allows the profile to be viewed and edited by the user themselves. This will allow them to create it before it's open to everyone, and also allows them to remove it from search results without having to delete it.

This will also involve expanding the search capability. Unify the public and private searches, providing the ability to display full public profiles, anonymized anonymous profiles, or full private profiles to users based on their logged-on status and the status of the profile. Additionally, exclude Hidden profiles, and change the text search to search region, professional biography, skills, employment history (employer, position, and description), and experience.


bit_span: add pop and consume

bit_span: add pop and consume
will be useful when decoding a DEFLATE stream
Change-Id: I4ba8659230270f8e571fe70444d9f5c629a888a3

nit:

    const auto distance = bit_offset_ + n;

I used to be all about const-ing everything but I recently read this which made me a bit doubtful about using it for local variables. I guess there's not much to say other than that there's trade-offs. Will go with const for now.
https://quuxplusone.github.io/blog/2022/01/23/dont-const-all-the-things/

Do either of these work?

At least the first one works on LLVM but neither works on GCC.

nit:

    const auto distance = bit_offset_ + n;

I used to be all about const-ing everything but I recently read this which made me a bit doubtful about using it for local variables. I guess there's not much to say other than that there's trade-offs. Will go with const for now.
https://quuxplusone.github.io/blog/2022/01/23/dont-const-all-the-things/

Yeah I favor const in general but I agree that it can often be unnecessary within function scope, especially if the function is small (like 10 lines). I'm influenced from when I wrote Rust and const is the default.
Herb also agrees on nonconst for locals:
https://github.com/hsutter/cppfront/wiki/Design-note%3A-const-objects-by-default

Do either of these work?

At least the first one works on LLVM but neither works on GCC.

std::views::iota is less typing and should work after updating boost-ut


Update readMe for IBM Bluemix ***

Hi Guys,
I am a Developer Advocate for IBM Bluemix, the Bluemix area on the readMe is out date, can you update it with below, please.
Thanks.
IBM Bluemix Cloud Platform

Create a Bluemix Account
Sign up for Bluemix, or use an existing account.

Download and install the Cloud Foundry CLI to push your applications to Bluemix.

Create a manifest.yml file in the root of your application.

applications:
- name:      <your-app-name>
  host:      <your-app-host>
  memory:    128M
  services:
  - myMongo-db-name

The host you use will determinate your application url initially, e.g. <host>.mybluemix.net.
The service name 'myMongo-db-name' is a declaration of your MongoDB service.  If you are using other services like Watson for example, then you would declare them the same way.

Connect and login to Bluemix via the Cloud-foundry CLI

$ cf login -a https://api.ng.bluemix.net

Create a MongoDB service

$ cf create-service mongodb 100 [your-service-name]

Note: this is a free and experiment verion of MongoDB instance.
Use the MongoDB by Compose instance for production applications:
$ cf create-service compose-for-mongodb Standard [your-service-name]'

Push the application
$ cf push

$ cf env <your-app-name >
(To view the *environment variables* created for your application)

Done, now go to the staging domain(<host>.mybluemix.net.) and see your app running.
Cloud Foundry Commands
More Bluemix samples
Simple ToDo app in a programming language of your choice
IBM Watson
Be sure to check out the full list of Watson services to forwarder enhance your application functionality with a little effort. Watson services are easy to use, it is as simple as a RESTful API call.
Here is an example of a Watson Toner Analyzer to understand the emotional context of a piece of text that you send to Watson.
Watson catalog of services
 AlchemyAPI - An AlchemyAPI service that analyzes your unstructured text and image content.
 Cognitive Commerce - Cognitive Commerce is a service provided by Cognitive Scale.
 Cognitive Graph - Cognitive Graph is a service provided by Cognitive Scale.
 Cognitive Insights - Cognitive Insights™ is a service provided by Cognitive Scale.
 Conversation - 	Add a natural language interface to your application to automate interactions with your end users. Common applications include virtual agents and chat bots that can integrate and communicate on any channel or device.
 Discovery - Add a cognitive search and content analytics engine to applications.
 Document Conversion - Converts a HTML, PDF, or Microsoft Word™ document into a normalized HTML, plain text, or a set of JSON-formatted Answer units.
 Language Translator - Translate text from one language to another for specific domains.
 Natural Language Classifier - Natural Language Classifier performs natural language classification on question texts. A user would be able to train their data and the predict the appropriate class for a input question.
 Personality Insights - The Watson Personality Insights derives insights from transactional and social media data to identify psychological traits.
 Retrieve and Rank - Add machine learning enhanced search capabilities to your application.
 Speech to Text - Low-latency, streaming transcription.
 Text to Speech - Synthesizes natural-sounding speech from text.
 Tone Analyzer - Tone Analyzer uses linguistic analysis to detect three types of tones from communications: emotion, social, and language. This insight can then be used to drive high impact communications.
 Tradeoff Analytics - Helps make better choices under multiple conflicting goals. Combines smart visualization and recommendations for tradeoff exploration.
 Visual Recognition - Find meaning in visual content! Analyze images for scenes, objects, faces, and other content. Choose a default model off the shelf, or create your own custom classifier. Find similar images within a collection. Develop smart applications that analyze the visual content of images or video frames to understand what is happening in a scene.
Click here for live demos of each Watson service.

Readme for the Bluemix section can also be found here: https://github.com/Twanawebtech/IBM-Bluemix-Deployment

@sahat

README has been updated.


[WIP] Add torsions for ring closure

This PR features/will feature a few improvements:

Addition of a torsion term to non-rotatable bonds to assist in ring closure.

Computation of internal coordinates via OpenEye (if this is faster, which it probably is)

Possible other speedups (such as omitting irrelevant atoms from the System that GeometryEngine uses).

I think you can use OEGetTorsion to measure torsions from the OEMol with omega-provided geometries.
Even easier would be to iterate over all isRotor=True torsions by simply using OEGetTorsions directly:
for torsion in OEGetTorsions(molecule):
    # use torsion.radians directly...

I think you can use OEGetTorsion to measure torsions from the OEMol with omega-provided geometries.

I'm aware--I'm also extending this to the other coordinate conversion stuff. I just decided to open the pull request before I finished--it's a WIP.

Even easier would be to iterate over all isRotor=True torsions by simply using OEGetTorsions directly:

Ok. Wouldn't I want isRotor=False though?

Ok. Wouldn't I want isRotor=False though?

Whoops!  Yes!

Ok, thanks, simplified the torsion collection considerably with that parameter. As I brought up here, it might be easier/less error-prone if the atoms in the OEMol that is extracted from the topology retain an old_index or something in their data. If the issue on openmoltools looks good, I can add that.

Hm, it doesn't look like this is helping much. Both the benzene-->biphenyl and erlotinib-->imatinib examples fail in the usual way. I did check to make sure the torsion was being applied, and that it didn't give ridiculous energy for the omega configuration, so I'm not sure what the issue here is.

Interesting! Let me take a look at the PR and see if I can spot anything.

Thanks!

Hm, I think I'm going to merge this and work on optimizing the GeometryEngine torsion calculation in the next PR.

Please do!

Ok.


[FEATURE]

Feature Description
A clear and concise description of what you want to happen.
Example(s)
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Possible Approaches or Libraries to Consider
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

hi @t9834241 let us know what you are after


Date object not passed correctly by emitClient

Fetching a DATE data type value from a MySQL table and sending it to the client via emitClient does not work. The Date object is converted to a [Object object] and is empty.
Client side:

Outputs: [23:04:48] DATE CREATED: [object Object]
Server side:

Outputs:
2020-05-29T22:00:00.000Z

test

test

The devs can decide whether to handle dates specifically or not, but the reason this is happening is that Date is not a primitive.
If you want to pass it between contexts, you'll have to serialize it on one side and parse it on the receiving end:
// server
const date = new Date();

alt.emitClient(player, 'sendDate', date.toJSON());

// client
alt.onServer('sendDate', dateJSON => {
    const date = new Date(dateJSON);

    // now you have a date object
});

As this is not an issue with the JS module but a suggestion for the alt:V Module SDK, refer to this issue https://github.com/altmp/altv-issues/issues/1118.


Client Id to send When requesting DHCP defaults to 'root'

causes error with some DHCP servers (including NetGear r6400 and my ISP)  [OpenWrt 18.06.4]

wan is setting up now
udhcp: malformed hex string 'root'

the fix was pointed out by [eduperez] WRT3200ACM no connect to ISP ... but i didnt know where to find setting in LuCi
for those coming later....
Network
Interfaces
WAN
Client Id to send When requesting DHCP  swap root with A1 or other HEXstring
<save & Apply>
dhcp connected almost immediately.

This is a browser bug, LuCI does not perform any kind of autocomplete.

I dont believe it is either browser or LuCi.
from https://openwrt.org/docs/guide-user/network/ipv4/start#protocol_dhcp
clientid | string | no | system default | Override client identifier in DHCP requests
it appears the system default is root , where some (presumably poorly implemented) DHCP servers expect a MAC address, or any valid Hex string. ie F2 works.
note that LuCi wont allow a blank ClientId, it reverts to [root]
I couldnt find this hack/workaround easily, so perhaps hope this helps others to do so now.
this issue hasnt raised for me previosly as the device was set with a static IP when acting as access point only.  same output on a different hardware running 18.06.2

There is no root default for client id, as can be seen here: https://github.com/openwrt/openwrt/blob/8852a09d85e5b0c2ffab55578a9624edb2fb6dd1/package/network/config/netifd/files/lib/netifd/proto/dhcp.sh#L54
If not set, -C is passed to udhcpc which instructs it to not set any ID at all. The root value is added by some browsers because they confuse the interface setting with a login form and helpfully add nonsense into the input field. It is certainly not LuCI which adds root as value.


apharo/remove node modules

chore: remove node_modules from git
chore: ignore node_modules
chore: run npm install

I think this might have broken the step, not sure how though. Getting this error on all my PRs for this step...
https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true
internal/modules/cjs/loader.js:818
  throw err;
  ^

Error: Cannot find module 'esm'
Require stack:
- /home/runner/work/_actions/blake-education/ember-asset-size-action/main/index.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:815:15)
    at Function.Module._load (internal/modules/cjs/loader.js:66[7](https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true#step:4:8):27)
    at Module.require (internal/modules/cjs/loader.js:[8](https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true#step:4:9)87:1[9](https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true#step:4:10))
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (/home/runner/work/_actions/blake-education/ember-asset-size-action/main/index.js:2:11)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:[10](https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true#step:4:11)27:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:[12](https://github.com/blake-education/district-ui-client/runs/8075926608?check_suite_focus=true#step:4:13)) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/runner/work/_actions/blake-education/ember-asset-size-action/main/index.js'
  ]

Based on some reading, I think it could be that we either need to keep node_modules here, or run npm install as part of the action or something. Am guessing the original creators has the node_modules as part of the repo for that reason :thinking:
https://turus.ro/posts/node-js-error-cannot-find-module-insert-module-name

Looks like javascript-based github actions do need node_modules as part of the project
The reason for that seems to be an ongoing debate on the example "hello-world-javascript-action" project
https://github.com/actions/hello-world-javascript-action/issues/12

Apparently the other alternative is to compile the node_modules into a single file with https://github.com/vercel/ncc and check that in to the project, which is what github does with all their first party actions
Seems interesting but I think I'll just revert this PR and call it a day

Oopsy - sorry for the trouble

Chuck me a PR and I'll approve it @Techn1x

Revert PR https://github.com/blake-education/ember-asset-size-action/pull/12
Follow up PR to update node_modules https://github.com/blake-education/ember-asset-size-action/pull/13

Oopsy - sorry for the trouble

no trouble at all! I learned stuff.


chore: CE-1275 fix startup errors

Addressed the errors in console on app startup.
Description
The redux-persist package uses non-serializable items that cause redux to throw warnings/errors. This PR suppresses them using the approach proposed in the Redux Toolkit documentation here https://redux-toolkit.js.org/usage/usage-guide#use-with-redux-persist .
Fixes # (issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Moved to new PR


When evading along bottom wall randomly step upward

This is because enemies follow horizontally and won't step on the mines, etc. This allows us to get out and around again.

Related: #2


Missing text and box when maximizing window + fullscreen

Problem:
When using rlImGui and using Raylib.MaximizeWindow(); and Raylib.ToggleFullscreen(); there are some text missing in imgui.
Setup:
rlImGui.Setup();
public override void Draw()
    {
        rlImGui.Begin();
        ImGui.ShowDemoWindow();
        rlImGui.End();
    }

public override void Update()
    {
        if (Raylib.IsKeyPressed(KeyboardKey.KEY_F11))
        {
           Raylib.MaximizeWindow();
           Raylib.ToggleFullscreen();
        }
    }

Showcase:
https://github.com/raylib-extras/rlImGui-cs/assets/72097037/622c19af-a70c-4f18-aaa1-8ca6984fbc22
Note:
This does not happen when only using Raylib.ToggleFullscreen(); so it's an issue related when using fullscreen in a window that is maximized.

Can you provide a complete example program that shows the problem and let me know what platform you are on?

I could also reproduce. You just need maximize a window, and then set fullscreen to it with   Raylib.SetWindowState(ConfigFlags.FLAG_FULLSCREEN_MODE);. Part of all the imgui windows does not show.
If the window is not maximised when going fullscreen, the issue does not happen.
Correct:

Wrong:

I found the problem, it was in both the C++ and C# versions of rlImGui and has been fixed in both places. Thank you for the report.


Fix pytest

My test run failed due to a missing js2py dependency, so I added that and froze the dependencies.

Thanks, wondering if adding just Js2Py is sufficient, or are all the other deps necessary to be in the requirements file?

@davidkpiano probably just Js2Py is sufficient. I typically use the pip freeze > requirements.txt, but I would be happy to revert and add just the one dependency if you prefer. I'll update the PR.

@davidkpiano probably just Js2Py is sufficient. I typically use the pip freeze > requirements.txt, but I would be happy to revert and add just the one dependency if you prefer. I'll update the PR.

Sure, can you revert it to just one dep? Thank you!

@davidkpiano done


[SPIKE] Investigate/Define Nightly Build Strategy

We need an opinion prepared on how we handle ODH nightly builds with DSPO/DSP.  Note that this should not be all-encompassing, but just a simple strategy documented (perhaps a few sentences maximum)
ACC/Questions to answer as a comment on this TASK:

What branch to build on
Where manifests come from
Will the current trajectory of nightlies being handled conflict with the DSP release strategy and cadence

The nightlies are now fully referencing upstream ODH manifests.

The sources file can be found here.

Builds are from the default branches (master/main)

Manifests are all pulled from opendatahub-io GH org

This should not affect release strategy, we should always make sure our manifests work with other components in their most latest ODH iteration (i.e. default branches). The only way it may conflict is if a dsp dependent ODH component (like odh dashboard, or elyra), does not put latest ODH content (i.e. changes expected to go in next odh upcoming release) in main/master of their respective branches

There is a cluster where nightlies are built and deployed to daily. Currently this cluster can be found here. Whether this will remain to be the case is unclear. If this link goes down, reach out to QE for the latest cluster.

Can these nightlies be deployed in other clusters?
Maybe. I tried but got blocked on requiring access to http://registry.stage.redhat.io, I think if we had an sa/token for this registry then it should work. Steps that got me this far:

Get images from jenkins builds here

Search for jobs prefixed with cvp-redhatopenshiftdatascience - odh-operator-bundle-container-v1.18.0 these are the nightly builds, the "1.18" is arbitrary/non-meaningful.

Allow insecure registry registry-proxy.engineering.redhat.com in your ocp cluster:

oc patch image.config.openshift.io/cluster --type merge --patch '{"spec":{"registrySources":{"insecureRegistries":["registry-proxy.engineering.redhat.com","registry.stage.redhat.io"]}}}' image

Add pull secret for http://registry.stage.redhat.io
??


Problem with References: only Auhtor present, no Year, no Details

Checking data on PREVIEW (CoL of 2021-12-13): https://preview.catalogueoflife.org/
Both World Ferns & World Plants have incomplete references. Only Author present, no Year, no Details.
https://github.com/CatalogueOfLife/testing/issues/4#issuecomment-993932102
https://github.com/CatalogueOfLife/testing/issues/3#issuecomment-993938228)
@mdoering, Geoff said, it is a backend problem: https://data.catalogueoflife.org/dataset/1141/verbatim?type=col%3AReference&col:ID=455468&termOp=OR

Hm, was the original data always so sparse?
There is now just an author and a citation column:
col:citation = Benth. 1837. Enum. Pl. Hueg. 53.
col:author = Benth.

If atomised fields are given these are trusted more, just as we do for scientific names.
So the resulting citation only has authors as the rest is missing.
If nothing but the author can be extracted I would say it is best to not include the author column at all but just use the citation string and nothing else.
Well, but this is weird. The Reference.csv file does contain other columns:
ID	citation	author	title	year	source	doi	link 455468	Benth. 1837. Enum. Pl. Hueg. 53.	Benth.		1837	In: Enum. Pl. Hueg. 53
The final ColDP format does not support year and source. Instead there is issued and containerTitle for the journal title (but not the volume etc, so it strictly does not match).
I will update the importer to also recognize year and source to be backwards compatible and map those columns to issued and containerTitle even though this is not strictly right.
@gdower it would be good to adjust your existing scripts to the final ColDP version.
We should really build a validator that primarily checks the syntactical structure of a ColDP archive and reports about unrecognized columns etc.

@dhobern I found that your Lepidoptera archives also still use last years reference format which was strongly based on ACEF.
The previous columns were:
ID,author,title,year,source,details,link
Could you update to the latest which is more aligned to BibTex and other reference standards?
https://github.com/CatalogueOfLife/coldp/blob/master/README.md#reference
Namely year, source and details are the 3 columns which have been replaced by others.

So, will I be correct if I keep it simple and change as follows?

year --> issued
source --> containerTitle
details --> page

I may be able to split details into issue and page, but I'll check whether the data is clean enough.

yes. Thats what I instructed the importer now also to do as a best guess when it faces these old ACEF terms in ColDP.
Obviously the "details" is a catch all thing which can contain all sorts of things, page(s), volume, issue. You probably know your data best to evaluate if that can be split into several terms. volume & issue would be great to ultimately locate an article, they are key if I understand Rod Page and others correctly.
issued is an ISO date that can be truncated, so all these are accepted:  1878, 1878-03, 1878-03-12

Fixed as issued, containerTitle, volume, issue, page for the Alucitoidea and Pterophoroidea datasets.

FIXED (preview 2022-01-12):


[SPARK-40544][SQL][TESTS] Restore the file appender log level threshold of the hive UTs to info

What changes were proposed in this pull request?
This pr restore the file appender log level threshold of the hive UTs from debug to info
Why are the changes needed?
Reduce the disk space occupied by sql/hive/target/unit-tests.log. After this pr, the file size reduced from 12G to 119M
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions

Merged to master.

thanks @wangyum


Va new

Cleaned up Basic Pipeline with no security
Add Adv Pipeline template with scan and metadata name got changed

can we change the name , advanced gives no clue why it is different , can we use security as its main purpose is align to the new dev sec ops security being the middle name , and be used for IBM Cloud for Financial Services , you could shorten it to just sec

Matt - thanks for inputs.
Idea behind this adv pipeline approach is

Today we are adding security task like va, trivia etc
Future we may add additional features that targets adv developer

@seansund - Pls chime if you want to change to "sec" instead of "adv"

Matt - thanks for inputs.
Idea behind this adv pipeline approach is

Today we are adding security task like va, trivia etc
Future we may add additional features that targets adv developer

@seansund - Pls chime if you want to change to "sec" instead of "adv"

I'm ok with renaming the pipeline to sec . If it becomes broader in the future we can always change it

As a conclusion. let me rename to sec and recheck in PR

@seansund

4 Pipelines are added as sec
Login to IBM Cloud explicitly required & namespace listing is required as part of TASK which could be due to the ROKS / VA / Security feature.
Buildah login supposed to work but for some reason which failed on my environment

@mjperrins  - FYI

@seansund  what does LGTM means ? Sorry acronym is new to me :-) anything i need to do from my side

Lets go to market :-)


Feature Scheduled Merges

Changelog

Add feature to merge pull requests labelled with the merge label

Codecov Report

Merging #50 (a89a1d1) into main (afce9e8) will increase coverage by 0.53%.
The diff coverage is 80.64%.

@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   83.36%   83.90%   +0.53%
==========================================
  Files          11       11
  Lines         511      528      +17
==========================================
+ Hits          426      443      +17
  Misses         60       60
  Partials       25       25

Flag
Coverage Δ

unittests
83.90% <80.64%> (+0.53%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/github/schedules.go
91.66% <80.00%> (+2.38%)
:arrow_up:

pkg/client/client.go
72.07% <100.00%> (+0.25%)
: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 afce9e8...a89a1d1. Read the comment docs.

/label enhancement

/approve

/merge


Question: Allocate Stock to Build

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
It isn't a bug, but a question.
There's any way to not allow a Stock Item to be allocated to multiples build orders?
Let's imagine that I have a part that is package as a reel and I have multiples reels of this part. When I allocate that part to a build order, on the Part Stock I can see a flag indicating "Stock item has been partially allocated" but, if I place another build order, while allocating the parts, I have no indication that this Stock item is already allocated.
The problem is, if I have two or more pick and places, I can allocated the same reel to multiples machines wich is impossible.
Steps to Reproduce

Go to  "Allocate Stock" in Build Order
Defining the Stock Item (no info if that Stock Item is already allocated to another build)

Expected behaviour
A warning flag saying that Stock Item is already beem allocated to another BO.
Maybe some checkbox in "New Stock Item" saying that this stock item cannot be allocated to multiples BO.
I think that we should manually define that because, if a part is not a reel, I can allocate to multiples BO.
Deployment Method

[X] Docker
[ ] Bare metal

Version Information
Version Information:
InvenTree-Version: 0.13.2
Django Version: 3.2.23
Commit Hash: ddd65cf
Commit Date: 17/01/2024
Database: postgresql
Debug-Mode: False
Deployed using Docker: True
Platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with
Installer: DOC
Active plugins: False
Please verify if you can reproduce this bug on the demo site.

[x] I can reproduce this bug on the demo site.

Relevant log output
No response

Currently this check is not implemented, but it would theoretical to implement it.


Annotation format

Hello,
Thank you for creating the dataset.
Sorry, if this seems to be a silly question.
Usually, in the yolov5 format, if there are multiple instances of the same class, the bounding box and class information is present in different rows right ?
But in your dataset, suppose if there are two people in a frame, there is just one row in which there is a class information followed by bounding box co-ordinates for two instances. Can I use this for my object detection directly, if I want to use it to train my yolov8 ?
Or
Is there a way I can convert these into correct yolov5 format ?

It was an issue with my notepad


SQL querying method called in two different methods makes server stops

What version of Go are you using (go version)?

$ go version
go version go1.12.7 linux/amd64

Does this issue reproduce with the latest release?
I believe go1.12.7 is the latest release at the moment
What operating system and processor architecture are you using (go env)?
go env
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ghacout/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ghacout/go"
GOPROXY=""
GORACE=""
GOROOT="/snap/go/4098"
GOTMPDIR=""
GOTOOLDIR="/snap/go/4098/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/ghacout/Bureau/GraphQL_GO/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build544526689=/tmp/go-build -gno-record-gcc-switches"

What did you do?

My problem is very specific, I was able to debug it but I am creating an issue because I want to understand why it did that and I was not able to find explanation on the internet.
Context
I created a golang webserver, in this server there are Data Access Object, which fetch data from a mySQL database. In my DAO there is a method to fetch products (GetProducts) this method calls two other methods (Properties & Populate) which both call method CustomFields (which fetch database columns names specific to the customer whom products we want)
There are multiple methods involved in this process :
DAO
func GetProducts () []Product {

	//properties is a list of database columns names
	properties := Properties()

        //Then using the properties I perform a SQL query on my Database and
        // it returns informations about products in form of SQL Rows in a variable "rows"

        //result is a list of products created with the informations provided by the DB
        result := Populate(rows)

	return result, nil
}

Methods
func Properties() []string {
	customFields := CustomFields()
        //Than we handle the customFields and returns list of database colmun names
}

func Populate(rows sql.Rows) []Product {
	customFields := CustomFields()
	//Than we handle the customFields in another way to return a list of products
}

func CustomFields() []string {
     //Get Custom fields will perform a simple SQL query on DB and return a list of custom fields name
}

Explanation
As you can see I removed a lot of code because it was just here to make you understand what's going on. There is no issue with the code and everything is working fine when I launch the server.
What did you expect to see?
A constantly working golang server
What did you see instead?
This issue is whenever is call my DAO a lot of times, about 10 requests per seconds my server is stopping. It's not dying and exiting, it's freezing and I have to relaunch it manually. This is not an infrastructure problem with the host machine or the mySQL database server.
It's due to Populate() & Properties() both calling the method CustomField(), because whenever I remove the CustomField() call in the Populate() method everything is fine. The server is working and can go up to more than 120 requests per seconds without trouble. After a lot of testing I am 100% positive that it doesn't come from anything else.
My question is : Why does calling a simple SQL query method in two different method is creating an issue ?
Thanks I hope I explained myself well and that someone will be able to help me understand what happened. If you have any questions feel free to ask

This sounds like a db locking issue. Do you close SQL.Rows? Do you defer tx.Rollback then call tx.Commit on happy path?

Hi, Thank you for your response
I do close sql.Rows like this :
rows, err := d.sqlProvider.dbConnection.Query(query, params...)
if err != nil {
	return c, err
}
defer rows.Close()

And since there are only "SELECT" sql query (i.e I'm only reading the database) I believed tx.Rollback and tx.Commit were not useful, was I wrong ?

That sounds fine then.
Without seeing more real code and sql, it will be hard to help.
My hunch is there is a bug somewhere in your code. This doesn't sound like a database/sql issue.

Hi and thank you again for trying to help me !
It's a bit complex because this code wasn't written by me and I don't want it to be publicly available without the writer's approval, however I can't have it right now.
But I can explain and show what it does to a certain extent.
//customFields() will return a slice of pointers (pointers to a struct named "customField")
func customFields() (CustomFields, error){

    var customFields CustomFields

    query := "SELECT ... FROM ... WHERE ... AND ... ORDER BY ... ;"

    //sqlProvider.Get() returns a dbConnection (*sql.DB for the sql package)
    rows, err := sqlProvider.Get().Query(query)
	if err != nil {
		return c, err
	}
	defer rows.Close()

	for rows.Next() {
		var c CustomField
		if err := rows.Scan(&c.ID, &c.Name); err != nil {
			return customFields, err
		}
		customFields = append(customField, &c)
	}

	return customFields, nil
}

//Get() method returns a *sql.DB
//sqlProvider is a struct containing informations about the SQL database (user, host, port, max OpenConnections... and a map[string]*sql.DB named "Connections")
func (s *sqlProvider)Get() {
        dbHost := s.Host
	if conn, ok := s.Connections[dbHost]; ok {
		return conn
	}

        //sql.Open will open a new connection with the sqlProvider informations
	conn, _ := sql.Open("mysql", ... )
	conn.SetMaxOpenConns(s.MaxOpenConns)
	conn.SetConnMaxLifetime(s.MaxLifetime)
	s.Connections[dbHost] = conn

	return conn
}

I hope you understood me and that you will be able to help me understand, thanks again !

I think I can help.
customFields looks fine in the query and and Scanning of rows.
The sqlProvider.Get looks odd, not the way I would recommend doing it. Ensure that s.Connections is guarded by a RWMutex. There isn't anything intrinsic to this code that would cause it to block. If you can repro, get into a blocked state then check how many open connections there are to MySQL and what state they are in.

Hi, it's a very interesting point and I absolutely agree that the map[string]*sql.DB should be protected by a RWMutex in case of write/read. It will be changed ASAP !
However It cannot be the source of my problem because in my case I only use one SQL connection (meaning I create it the first time then reuse the same one all the time). I checked with metrics on mySQL server and there is only one open SQL connection as expected.
Thanks again for your help !

The symptoms sound like out of available connections and the engine blocking waiting for one to free up.
While details are missing, it looks to me like your code is opening a db connection in GetProducts and passes the rows to populate. This connection is still open when subsequently calling customFields which is in need of an available database connection to proceed. Until your single connection closes it'll block, but since its single threaded, it has no hope of that. Even if killed from the server, the client pool of one connection will think it's still in use.
You may need to use a connection pool of multiple connections instead of a single one from the map tied to dbhost.
In addition, close the connection, or release to the pool, as soon as possible.

Closing old issues that still have the WaitingForInfo label where enough details to investigate weren't provided. Feel free to leave a comment with more details and we can reopen.


Save/Load operations not fully supported

Keeping track of what's needed

[ ] Cannot save circuit without vtree (workaround: use infer_vtree to do that after fixing its limitation #47)
[ ] Cannot save circuit that is not struct decomposable, need to choose a format for those cases.
[ ] Cannot save CNF/DNFs (can only load them)
[ ] Save a Prob/Logistic/Regression circuit without vtree
[ ] Saving/Loading of regression circuits (now mostly support logistic circuits only)
[ ] Save/Load mixture models, (eg from strudel). See https://github.com/Juice-jl/ProbabilisticCircuits.jl/issues/28 for suggestions.

For the load/save of Strudel (mixtures), one suggestion is that save all related files and metadata in a single Zip file (with appendix .strudel). We need the structure, the vtree, the mixture weights, and paramters. (maybe more). One option is using ZipFile.jl. Also see Juice-jl/LogicCircuits.jl#12

Also can't save plain circuits.
Also save for many other things is not available, probably need a save option for everything we load from file:

vtee
cnf
plain circuit
structured circuit
......

One more issue to fix, for save methods some take the file name first some take the file name second, should make them be  consistent, for example:
save_as_sdd(name::String, circuit, vtree)
save_vtree(vtree::PlainVtree, file::AbstractString)
save_as_dot(circuit::LogicCircuit, file::String)
....

Also related to #40

The first 3 check boxes above are resolved in https://github.com/Juice-jl/LogicCircuits.jl/pull/99
The remaining ones should be moved to https://github.com/Juice-jl/ProbabilisticCircuits.jl


Make 104 keyboard firmware

Make 104 keyboard firmware, add PIN pin under custom_board.h and pin_assign.h file, make firmware, add some pins and no response.
NRF52 does not have a CAPS/NUM LED pair?

I'm not supposed to decode this.

 You can solder your own though.
Assuming "CAPS/NUM LED pair" is about indicator leds (there are usually three of those), I'm really not sure, it may be lost somewhere in the nrf52 fork it could need reading through the code. I haven't really used those.

I think you are right about the nrf52 fork losing caps/num indicator support, as it didn't work for me when I tried displaying those led status on the crkbd OLED a few days ago.


Add workflow for SemVer checks

We should bump cargo.toml of specific changed pallet, runtime, and spec / implementation versions when new changes are being merged to the master branch

this could be useful https://github.com/rust-lang/rust-semverver

related https://github.com/galacticcouncil/Basilisk-node/pull/209

already done in parachain


fix: Subscriber Addition Request

Adapted the subscriber addition request and the response to the new Novu api.
Ticket - https://github.com/novuhq/novu-java/issues/29
This is my first pull request ever, so I'm not sure whether pulling on the master was the right thing to, or if I should have created a new branch for it.

@mayorJAY sure, but at the moment, the method for removing a subscriber, receives in input the same SubscriberAdditionRequest class:
removeSubscriberFromTopic(SubscriberAdditionRequest request, String topicKey);
So, the update should have fixed it aswell.
If you want, for code clarity, I can split them and create a SubscriberRemovalRequest, so each class will have its own responsibility.
I'll also check if the SubscriberRemovalResponse is working as expected, but I probably won't be able to work on it before Monday

@mayorJAY sure, but at the moment, the method for removing a subscriber, receives in input the same SubscriberAdditionRequest class:
removeSubscriberFromTopic(SubscriberAdditionRequest request, String topicKey);
So, the update should have fixed it aswell.
If you want, for code clarity, I can split them and create a SubscriberRemovalRequest, so each class will have its own responsibility.
I'll also check if the SubscriberRemovalResponse is working as expected, but I probably won't be able to work on it before Monday

@luzzetti I agree with you. It looks good so far


Not enough space between search box and title

See screenshot.

Working on this.

I suggest to keep it on the right side of the navbar & decrease the Search input's width... Until we reach the small screens, at that point; Keep it below the links. (With the navbar background)


Enchants

My factions players has been reporting 2 enchants not working
Blessed hasnt been working
Inquisitive hasnt been working
please fix theres asap! Thanks for all of the hard work your doing! and happy new year!

Hello, ill look into it but I have tested inquisitive many times before and it was fine. I am sure blessed should work also. The chances may be a little low so they mite not notice it happen.


TypeScript lib checking errors using this module

I'm using the latest TypeScript version 3.5.3 and I'm getting the following errors:
node_modules/builtin-modules/index.d.ts(12,40): error TS1005: ',' expected.
node_modules/builtin-modules/index.d.ts(12,46): error TS1005: ',' expected.
The import I'm using is import * as builtinModules from 'builtin-modules'; and the option skipLibCheck is disabled so the compiler is checking for correct typing files.
The line it complains about is the following:
declare const builtinModules: readonly string[];
It looks like the reason is that the const is additionally declared as readonly which TypeScript apperantly doesn't like. Removing the readonly fixes the error for me.

We have type tests that are passing with the latest TS: https://github.com/sindresorhus/builtin-modules/blob/master/index.test-d.ts

Oh, looks like it was my bad. I was running tsc manually which was using the global version which was still on 3.3.3333. Updated it and it worked again.
Sorry for the inconvenience.


BLE - Nimble and blob create service

BLE - Nimble and blob create service
Description
Adding Nimble drivers for esp - deactivated for now
Adding blob create service support.
Mode supported on Nordic only
Adding versionning for BLE
Adding lengh parameter for short local name on ble hal
Nordic now support both blob option and separate attribute creation.
Tested:
MQTT demos and ESP and Nordic
BLE tests on ESP and Nordic
Checklist:

[x ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Failed to load source map for

Describe the bug
Errors for like the following are appearing in console when running vite
04:20:17 [vite] Failed to load source map for /node_modules/.vite/vue-router.js?v=931fb4d7.
04:20:24 [vite] Failed to load source map for /node_modules/.vite/axios.js?v=32dedaba.
04:20:24 [vite] Failed to load source map for /node_modules/.vite/mosha-vue-toastify.js?v=32dedaba.
04:20:24 [vite] Failed to load source map for /node_modules/.vite/vuetify_lib_components_VBtn_index_mjs.js?v=32dedaba

The only problem since this started happening is Vuetify icons do not work? - This was resolved I am not sure even how
Reproduction
Attempting this in another project, can't reproduce this currently
Attempted to reproduce this however was unable. This only happens in my big project.
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (12) x64 Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz
    Memory: 18.96 GB / 31.94 GB
  Binaries:
    Node: 14.18.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.21.1 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 8.4.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 99.0.4844.82
    Edge: Spartan (44.19041.1266.0), Chromium (99.0.1150.46)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-vue: ^2.2.0 => 2.2.4
    vite: ^2.8.0 => 2.8.6

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Please try Vite 2.9 beta too if that fixes this issue, as that contains some improvements to prebundling. Otherwise we'd need a repro to debug this.

I will try it and reply with the result.


Inconsistent name rules across different entities

idcs-ddeba14a21d74e72baf5a26c65900c12-identity-c9dev2-oc9qadev-com is a valid name for at least service entries and destination rules, but creation of a Gateway with the same name fails with a generic name validation error. Shortening the gateway name to a 15 characters DNS-format string solves the problem, but breaks the idea of simple correlation between ServiceEntry, DestinationRule, Gateway and VirtualService
Error message (I'm using the snowdrop Java client for Istio):
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: POST at: https://<IP_ADDRESS>:6443/apis/networking.istio.io/v1beta1/namespaces/dx4c-prod/gateways. Message: admission webhook "validation.istio.io" denied the request: configuration is invalid: invalid gateway name: "idcs-ddeba14a21d74e72baf5a26c65900c12-identity-c9dev2-oc9qadev-com". Received status: Status(apiVersion=v1, code=400, details=null, kind=Status, message=admission webhook "validation.istio.io" denied the request: configuration is invalid: invalid gateway name: "idcs-ddeba14a21d74e72baf5a26c65900c12-identity-c9dev2-oc9qadev-com", metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=null, status=Failure, additionalProperties={}).
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
All names that are valid names in Kubernetes should be accepted on Istio entities. Where the naming has to be more strict compared to base Kubernetes, the validity rules for Istio entities should at least be the same.
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
istioctl version --remote
client version: 1.6.2
cluster-local-gateway version:
telemetry version: 1.6.2
pilot version: 1.6.2
data plane version: 1.6.2 (20 proxies)

How was Istio installed?
via istioctl
Environment where bug was observed (cloud vendor, OS, etc)
Kubernetes 1.18.4 installed via kubeadm

I confirm this is happening, and can be reproduced with istioctl validate -f <filename>.  Istio insists the Kubernetes name of a Gateway be a valid RFC-1123 hostname.  This does not seem correct.
Our documentation https://istio.io/latest/docs/reference/config/networking/gateway/ does not mention this limitation, nor discuss the Kubernetes name of the Gateway resource at all.
The code is in pkg/config/validation/validation.go.  I don't know if there are any circumstances in which this check is needed.

We do not want dots, but length can be long I think: https://github.com/istio/istio/issues/13211


Clarify polyfilling and browserlist usage documentation

Hi,
The docs about supported browsers is not clear.
After reading it i still dont understand the following :

If the polyfills are not imported, what the browserlist field in package.json is used for ?
If i need to support ie11 (and Array.includes for example), what do i need to do exactly ?
Is @babel/preset-env used to automatically require polyfills ?
Moreover the docs from @babel/preset-env (included with preset react app) indicates to import @babel/polyfill which is now deprecated. react-app-polyfill/stable.js requires the correct package core-js/stable but is browserlist still used using this file ?

Thanks for your help

I ran to a similar issue myself not too long ago and I think first and foremost it would make sense to fix the instructions on Babel's end first

I ran into same issue. Actually I want to support IE11 and from what I understand I have to import import "react-app-polyfill/ie11"; at the top of the index.js file what I do. However, checking polyfill's code I don't see any code for Array.prototype.includes which is not available in IE11...

Hi @lukejagodzinski, you'll also need to add react-app-polyfill/stable as stated in the docs. The ie9 and ie11 entrypoints only contain language features needed for CRA itself

@petetnt yep I've noticed that. But instead of polyfilling everything I've just polyfilled features that I needed using core-js and it works. Would assume that when I use browserslists it should automatically polyfill everything that's needed and also detect what features I'm using and only polyfill them. That would be ideal scenario.

I just tried to modify the browserlist config to polyfill for older version of Firefox (53), and had to do:
src/index.tsx
// First line in file
import "react-app-polyfill";

// ... other stuff

But, unless I'm reading the code wrong, it seems that the current setup should automatically polyfill based on the browserlist config aka:

it should automatically polyfill everything that's needed and also detect what features I'm using and only polyfill them

See https://github.com/facebook/create-react-app/blob/v3.1.1/packages/babel-preset-react-app/create.js#L78-L92
      (isEnvProduction || isEnvDevelopment) && [
        // Latest stable ECMAScript features
        require('@babel/preset-env').default,
        {
          // Allow importing core-js in entrypoint and use browserlist to select polyfills
          useBuiltIns: 'entry', <--- THIS LINE HERE
          // Set the corejs version we are using to avoid warnings in console
          // This will need to change once we upgrade to corejs@3
          corejs: 3,
          // Do not transform modules to CJS
          modules: false,
          // Exclude transforms that make all code slower
          exclude: ['transform-typeof-symbol'],
        },
      ],

Would assume that when I use browserslists it should automatically polyfill everything that's needed and also detect what features I'm using and only polyfill them. That would be ideal scenario.

@lukejagodzinski yes that is technically supported in @babel/preset-env by setting useBuiltIns: "usage", but it can lead to subtle bugs that you may not expect when usage doesn't detect the correct usage and it fails to run in a specific browser. Instead we rely on browserslist to polyfill everything needed in your target browser when you choose to import "react-app-polyfill/stable". More advanced usage we recommend that you import the individual polyfills that you need. This will result in a smaller bundle as well.
@ksrb it doesn't look like you've followed the directions over at https://github.com/facebook/create-react-app/blob/master/packages/react-app-polyfill/README.md#polyfilling-other-language-features. Try following that and see if it works for you.

Can someone answer the questions raised by the topic starter one by one?

Can someone answer the questions raised by the topic starter one by one?

Is it possible to deactivate the browserlist config altogether?


Use of aliased tables in functions generates incorrect SQL

Problem
Using a table as an argument to a function works correctly, until the table is aliased.  Consider the following code:
products = Table('products').as_('p')
test_query = Query.from_(products).select(fn.Count(products))
print(str(test_query))
# SELECT COUNT("products" "p") FROM "products" "p"

Expected:
SELECT COUNT("p") FROM "products" "p"

Passing a table to a function is a supported operation in PostgreSQL and passes the row data, it can be used for functions such as SELECT custom_convert_to_json("p") FROM "products" "p" (which is where I originally encountered the issue).  Note that SELECT COUNT("p".*) FROM "products" "p" is also an accepted syntax which does not currently seem possible to generate with PyPika.
Workaround
Removing the table alias works fine, but sometimes isn't an option due to name conflicts.  In those cases, creating a fake "table" with the same name as the alias also works fine.  This is what I'm doing for now.
products = Table('products').as_('p')
fake_p_table = Table('p')
test_query = Query.from_(products).select(fn.Count(fake_p_table))
print(str(test_query))
# SELECT COUNT("p") FROM "products" "p"

I took a quick look in the source for converting function parameters and it seems to me like changing this behavior isn't straightforward without either adding special case handling when isinstance(Table) (which would have to be done for all functions), or modifying the str() behavior of aliased Tables, which would of course have other implications.

Hi @davidpurser,
Besides the table alias you can also pass *, any literal, any field and other terms in most RDBMS.
Have you tried passing anything else? This should work fine for your example:
products = Table('products').as_('p')
product_id = Field('product_id', table=products)
test_query = Query.from_(products).select(fn.Count(product_id))

note: using a field like the above will also support specified counts from any side of a join if you are not doing an inner join.


Designer trying to implement - No images appearing

I'm struggling with getting instafeed.js to work correctly. So far I have this...

I used elfsight to generate my accessToken, so I'm just assuming that should work. Other than that I'm not sure what I'm not doing correctly.
I'm not getting an error message or anything.
Any help would be appreciated

dont use github as your js source
download the file and include yourself

@baboles you're missing the clientId parameter

@baboles example code using target
          $(document).ready(function() {
              var feed = new Instafeed({
                clientId: "*****************",
                target: "instafeed",
                get: "user",
                userId: *****************,
                accessToken: "*****************",
                template: ''
              });

              feed.run();
          });

@kitwalker12 I used your example code after not having much luck, and I get a 404 not found error on the resource its trying to pull. Any ideas? Obviously the code below has the clientID, userID, and accessToken edited out.
`
`

@MikeTheVike sorry. not sure. It probably has something to do with incorrect clientID or accesstoken.

@kitwalker12 it turns out the userID is a number I had to find, and not username of the account like I thought. Added the number and it worked for me. Thanks!


CSS Variable Support

You know what I'm talking about anyway, but CSS variables aren't supported at the moment. It seems that inspecting code with variables causes the tab to crash in Chrome as well, so I was finding it hard to test and add this feature.
http://dabblet.com/gist/3264403 is an example.
In short, the two things needed to be prefixed are:
var-bgcolor: green;

and
background: var(bgcolor);

The difficulty (if it is one!) in comparison to other CSS is that the var-bgcolor can of course end in anything, so it's a case of identifying things that start with var-, rather than picking up the whole property.

I think specification changed, the regex should be
...return css.replace(/(?:^|\{|\s|;)--(?:[\w-]+)\s*:\s*[^;}]+|(\s|:|,)var\s*\(--([\w-]+)\)/gi, function(......
var declaration = $0.match(/(^|\{|\s|;)--([\w-]+)\s*:\s*([^;}]+)/i);

Pull request?


refactor(interactive): Generate Stored Procedure Config Automatically in Compiler

What do these changes do?
as titled.

Related issue number

Fixes

Codecov Report

Merging #3140 (11203db) into main (8950371) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3140   +/-   ##
=======================================
  Coverage   42.29%   42.29%
=======================================
  Files         101      101
  Lines       10907    10907
=======================================
  Hits         4613     4613
  Misses       6294     6294

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 8950371...11203db. Read the comment docs.


Upload attachments and user avatars as ghosts

I'm not going to have time to fix the tests any time soon. It'd be great if anyone else had an opportunity to pick this up.


discovery: Hook up service name source reporting

What does this PR do?
Report the service name source as already determined by the current code.  This indicates whether the service name source was "provided" from a user-provided DD_SOURCE or similar ("provided"), or was "generated" (by the agent or by the injection library).

Motivation
https://datadoghq.atlassian.net/browse/USMON-1164

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

one question - how do we distinguish between DD_SERVICE set manually by the customer, and DD_SERVICE set by the auto-injector?

The auto-injectors sets another variable which indicates it injected DD_SERVICE:
https://github.com/DataDog/datadog-agent/blob/dccd0de82cede2cdf4dcc588966ef5eadeae6cac/pkg/collector/corechecks/servicediscovery/usm/service.go#L135

/merge


Doesn't allows to create Taxable accounts with 'No Tax'.

To Reproduce
Steps to reproduce the behavior:

Go to 'Accounts Page'.
Click on '+Account'.
Select DI/SA or II in Account Type field.
Click on 'Add Tax'.
Click on Save/Add tax btn.
Now, Click on Save Changes

Screenshots

its same on live version


CycleCloud Shared Service

Resolves issue #1599
What is being addressed
Add a CycleCloud shared service and documentation.
Remove autoblock from Nexus proxied repos as this blocks when there are transient issues leading to failed deployments.

/test-force-approve

/test-force-approve


Support turbo

Hey, as promised, a draft PR! Comments welcome :)
Notes

I just converted the map config plugin to a plain ol' function
added support for ['plugin-name', { ...options }] syntax
split the plugin resolution into two types, the 'input' phase contains the unresolved imports
added a method to resolve these imports
allowed overriding the providerImportSource. I believe this will be mandatory when using turbo because we can't rely on webpack aliases
I have not ported / modified the search plugin

Questions:

how do we communicate to the user that if using turbo you need to have serializable options? the types here will be funky. maybe 'createMDXTurbo' that just wraps the normal fn and adds the extra constraint?
right now this syntax only supports default imports. I propose we ship it and come back to it if anyone needs that feature.

Still need to write docs but will come back to that once these Qs are resolved.
Testing
Here is a page on my deployed site https://github.com/arlyon/litehouse/blob/main/site/content/docs/plugins/index.mdx?plain=1#L21-L22 that uses both a remark plugin and a rehype language.
You can validate the webpack side still works here https://litehouse.arlyon.dev/docs/plugins and if you'd like to clone and build the repo you should be able to do that fairly trivially but here is also a screenshot.

how do we communicate to the user that if using turbo you need to have serializable options? the types here will be funky. maybe 'createMDXTurbo' that just wraps the normal fn and adds the extra constraint?

This is fine, I think mentioning it on the docs is enough (since types can be hard to deal with).
But apart from passing remark and rehype plugins. For Shiki (the syntax highlighter), people might want to add their own transformers which also needs to be a function, the eco-system isn't pretty friendly for this

right now this syntax only supports default imports. I propose we ship it and come back to it if anyone needs that feature.

Agree, maybe I can make the support for named imports like remark-math/something{name} later.

TY for the quick feedback. I will have a think about shiki over the weekend. I think the answer will be that user-defined transforms as inline functions are a no-go. You'd need to put them in a different package in your repo and import them.

Could we throw a warning in that case? Or mark the support as experimental? My thinking is that we could get support out for the folks not using transformers (I'd have to assume the majority of folks) and look forward to the wider support in the future.

Before Turbopack goes stable, I'll mark it as experimental. Maybe we can check the passed options in development mode, then throw the warnings if unserializable data detected.

Just an idea: How about making a configuration file for the loaders, e.g. mdx.config.ts and bundle that file on-start?
After bundling the configuration file, we can call eval it (theoretically) on the runtime of Turbopack and obtain the options for MDX loaders. I'm not sure if it against the design of Turbopack, but I think having an easier migration experience would be more motivating for people to switch to Turbopack.

I cleaned some code and added the isSerializable function for checking the MDX options

In Next.js, when you able the --turbo it also sets process.env.TURBOPACK = '1'. So instead of the default value for experimentalTurbo being true should it be Boolean(process.env.TURBOPACK)?

Hey @fuma-nama, do you have strong opinions regarding the syntax? Is the default export syntax I proposed ok in the mean time? Regarding https://github.com/fuma-nama/fumadocs/pull/468#issuecomment-2169079748 the intention is to have closures work ootb so we should be able to remove this experimental syntax eventually.
If you are on board I will rebase it and we can get it in :)

My only concern is that the current setup still lacks flexibility, since most remark/rehype plugins are not designed to work on Turbo, people might want to pass unserialisable options.
I'm taking reference from other similar things like Content Collections (or Contentlayer). If the loader can also bundle the configuration file with Turbopack, we might be able to "evaluate" the config file in runtime, and get unserialisable options working with our MDX loader.
From what I understand, we cannot pass unserialisable options to loaders, because there's a Rust layer between next config and loaders. But if it's a JavaScript loader evaluating another JavaScript config, it should be possible.
Sorry if it's too much change, highly appreciate your efforts to improve the build system ;)

Now I'm working actively on Fumadocs MDX, a few news:

the new MDX v10 can support Turbopack
Content Collections also works with Turbopack

I'm trying to use esbuild to bundle the configuration file (source.config.ts), and load it from MDX loaders. This makes it possible to load unserializable options.
If Turbopack has its own functionality to load TypeScript file/config (e.g. a bundle() function), I'll willing to switch from esbuild to Rust-based solutions.

Closing in favour of https://github.com/fuma-nama/fumadocs/pull/778


New Services to be added to ServiceRequestCausesLoggingRequest

ForwardingList and CONFIGfile:
The following service need to be added to the forwarding ServiceRequestCausesLoggingRequest:

/v1/dispose-remainders-of-deregistered-application
/v1/inform-about-preceding-release
/v1/update-client-of-subsequent-release

Changed as proposed and merged into 2.1.1_spec branch


Docker image-only deployments

This PR will change our deployment strategy from having users installing terraform and ansible, cloning the repo, and then get into configuration, to docker run. As of right now, the docker image that I'm using for testing is on my personal docker hub. I would like folks to review this and identify any gaps.

@sehqlr Let me know when this is ready to review/test. Also, the commits should be squashed to a few logical commits when we are closer to merge time.

As soon as I can get a green check from travis, this will be ready for review and squashing.

OK, so I can't get a green light from Travis, because of the distributive checks issue, but other than that this is ready for review. Should I squash the commits now, or after review?

Broad thought: This adds a lot of files to the root directory. Can we put them in a subdirectory?

Second thought: This creates/modifies a bunch of scripts, I'm left wondering why more of them couldn't be in the same script.

I'm not a big fan of renaming the testing/*.tf files to their names without extensions. What's the reasoning there?

Closing in favor of #1289


Add lint rule to ensure use of capture action when numerical transaction registers are used

The numerical transaction registers (TX.0 through TX.9) are only filled when the capture action is used. If a rule uses any of the numerical transaction registers but does not use the capture action, it will see values set by other rules.
We should have a lint rule that prevents this situation from occurring.

@airween? ;)

@airween? ;)

Sure! :)
You mean something like this:
SecRule REQUEST_HEADERS:Referer "@rx ^[^#]+" \
   "id:932205,\
    ...
    capture,\
    ...
    chain"
    SecRule TX:0 "@rx ^..." \
        "capture,\
        t:none,\
        chain"
        SecRule TX:1 "@rx /" \
            ...
                "

so in this case the chained rule before uses TX:N must contain capture, right?
What if the rule is not chained, but uses TX.0, for example in logdata? I think the capture needs in that case too.
   ...
   logdata:'Matched Data: %{TX.0} found within %{MATCHED_VAR_NAME}: %{MATCHED_VAR}',\
   ...

Yes. The capture action is always required. It can be misleading to look at the log because sometimes the desired information ends up in the variable by accident, simply because a different rule set the value previously.

@airween Any progress here?

Yes, I already made it. I need to test it and make a last check - I can send that soon.

See #3825.

Closing via #3825.


proxy_check  do nothing ?

    def proxy_check(self):
        assert self.proxy and self.webdriver, 'Scraper instance needs valid webdriver and proxy instance to make the proxy check'

        self.webdriver.get(Config['GLOBAL'].get('proxy_check_url'))

        data = self.webdriver.page_source

        if not self.proxy.host in data:
            logger.warning('Proxy check failed: {host}:{port} is not used while requesting'.format(**self.proxy.__dict__))
        else:
            logger.info('Proxy check successful: All requests going through {host}:{port}'.format(**self.proxy.__dict__))

proxy_check do nothing, if proxy is not vaild, programe still use it.

Fixed. The scraper instance will not run when the proxy isn't working.


Should not depend on entire rails

Only depend on parts of rails that is necessary. Only activerecord I guess?

Only depend on parts of rails that is necessary. Only activerecord I guess?

Yes, indeed.

This commit fixed it.

And activemodel?

And activemodel?

I only specified activerecord because it depends on activemodel.

but of course 🤦♂


Refactor the ActiveAck implemenation to simplify testing

This is based on discussion in #4641 and to simplify adding test for changes to be introduced in #4632
Description
This PR refactors out the logic related to active in a separate class which enables implementing unit test for the same. This also abstracts out the user event flow such that it can be tested independently of its default configuration
There is no functional change in this PR
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4658 into master will decrease coverage by 4.7%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master    #4658      +/-   ##
==========================================
- Coverage   83.12%   78.41%   -4.71%
==========================================
  Files         184      185       +1
  Lines        8438     8442       +4
  Branches      591      593       +2
==========================================
- Hits         7014     6620     -394
- Misses       1424     1822     +398

Impacted Files
Coverage Δ

...pache/openwhisk/core/invoker/InvokerReactive.scala
79.46% <100%> (+21.61%)
:arrow_up:

...he/openwhisk/core/invoker/MessagingActiveAck.scala
53.84% <53.84%> (ø)

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-96.17%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-94.74%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedListener.scala
0% <0%> (-86.67%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-76.48%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-74.08%)
:arrow_down:

... and 32 more

Continue to review full report at Codecov.

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


Support "account" lockout for Patient Links with consecutive unsuccessful verification attempts

CDC has a requirement that 5 consecutive unsuccessful login attempts disable an account for at least 15 minutes. We added a policy in Okta to do this for provider & administrator accounts, but this is something that will need to be implemented for patient “accounts”
Since patient accounts are meant to be used twice (once to populate the patient form and then again to submit), a quick and dirty solution could be to add a counter  to the table storing form IDs:

The counter would need to be incremented on each lookup and (optionally?) reset to 0 on each successful use.
If the counter reaches 5, all access attempts using that form ID would fail.
No cooldown period implementation; if the ID gets disabled, the patient would need to request a new link (which would given them a new form ID)

Relates to #921


Update handling-concurrency-with-the-entity-framework-in-an-asp-net-w…

…eb-application.md
Changing year from 999 to 1999 in description of result.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@tdykstra  please review

@kscelfo Thanks!


fix current-tab-indicator offset in firefox

before:

after:

[x] chrome works the same, still

can you please create an issue and link it to this PR, please? same for the other one as well.


Fix documentation for Promise.using

Some of the docs involving Promise.using were slightly mangled when migrating documentation from 2.x. This fixes #874.

:+1: This is a really nice feature that is very hard to use with the documentation currently on the website.

Thanks.


Yet another "nothing covered" (with gulp-mocha 4.x)

I'm using the "gulp-mocha" plugin to run my unit tests (which are ES6 modules processed by Babel). Everything worked fine until I decided to upgrade this plugin to the latest version (v4.0.0).
Now my coverage report is always empty. It seems that this new version of "gulp-mocha" does not play well with your plugin.
Any idea how I can restore the coverage report? Do you know any alternative?
I don't wish to downgrade the "gulp-mocha" plugin.
Here is my "gulpfile.js" (see the test tasks):
https://github.com/cedx/akismet.js/blob/master/gulpfile.js

+1.
Both running istanbul from CLI or downgrading gulp-mocha to 3.x.x fixes the issue.

gulp-mocha 4 doesn't run mocha in the same process (it spawns the mocha cli on the command line). As such, istanbul.hookRequire() won't work anymore.
I'm not sure how given the new gulp-mocha workflow you'd get this setup to work at all.
You can write the covered files to disk so mocha read them, but then I'm unsure how you'd go about retrieving the coverage data.

Running istanbul from CLI [...] fixes the issue.
...
gulp-mocha 4 [...] spawns the mocha cli on the command line

The more I use Gulp, the less I can take advantage of its streaming possibilities :disappointed_relieved:

To be honest with you, I'm really rarely using gulp anymore. I find it easier to use tools with more built in capacity - like jest for testing, webpack for compilation, etc.

Finally fixed my issue by:

replaced gulp-mocha plugin by a direct call to the mocha CLI
replaced gulp-istanbul plugin by a direct call to nyc (aka the Istanbul CLI)

I still want to thank you @SBoudrias : your plugin has been very useful to me for a long time. Do I keep this issue open?
French PS (off-topic) : idem de mon côté Simon, je me rends compte qu'au fur et à mesure mes builds Gulp ne font plus grand'chose qu'appeler des binaires CLI... Je songe sérieusement à me contenter des scripts NPM pour lancer mes opérations de build, Gulp n'étant devenu qu'une surcouche presque inutile. Même si je ne vais vraisemblablement plus utiliser gulp-istanbul, encore une fois merci pour les efforts consacrés à ce plugin (je sais que s'investir dans l'Open Source est très chronophage, et ne rapporte pas grand'chose... à part l'estime de ses pairs).

I close this issue since the problem is not really related to this project.

This issue is still referenced in the readme file, would be nice to keep it open or at least refer to the open issue in gulp-mocha module.

Why is this still an issue?

Locking conversation to prevent being spammed.

gulp-istanbul cannot work with<EMAIL_ADDRESS>Architectural changes made to gulp-mocha made it stop using files inside the pipe, so it will run your tests, but doesn't work in a gulp way anymore.
I do not use gulp anymore, and suggest you to drop it too. Alternative suggestions: use jest or mocha and nyc directly.
If someone wants to maintains this project, you can reach out to me.


Debugging Plutus code

It would be nice to have debugging facilities for "Plutus code". Here I mean both Plutus Core and Plutus Tx.
I think we can give a reasonable debugging experience with the following pieces:

A step-debugger for PLC evaluation

The CEK model lends itself quite well to this, see some experiments of mine

Good source maps from PLC code back to the Plutus Tx that was written by the user

I have the beginnings of this in the form of Provenances, but it's very preliminary.

A sensible way to inspect values in environments

Really we would like to print them as the corresponding Haskell value would be printed/navigated. I think we should be able to do this if we can get string support or similar via dynamic builtins.

Since script execution is deterministic, we should be able to always reproduce a script execution failure locally in a way that you can debug.

I'm curious as to how we want to handle the step-by-step debugger in
particular. Maybe GTK?
Are there any example debuggers/tracers already in industrial use we
could look at for reference? I remember using traces in Eclipse (for
Java) aeons ago... What is the state of the art for these things?
On 2/12/19 2:42 PM, Michael Peyton Jones wrote:

It would be nice to have debugging facilities for "Plutus code". Here
I mean both Plutus Core and Plutus Tx.
I think we can give a reasonable debugging experience with the
following pieces:

A step-debugger for PLC evaluation
o The CEK model lends itself quite well to this, see some
experiments of mine
https://github.com/michaelpj/abstract-machine-debuggers
Good source maps from PLC code back to the Plutus Tx that was
written by the user
o I have the beginnings of this in the form of |Provenance|s,
but it's very preliminary.
A sensible way to inspect values in environments
o Really we would like to print them as the corresponding
Haskell value would be printed/navigated. I think we /should/
be able to do this if we can get string support or similar via
dynamic builtins.

Since script execution is deterministic, we should be able to always
reproduce a script execution failure locally in a way that you can debug.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/plutus/issues/629, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMpUzghDHiMnj0wNG4lZNzq-MWd_hZJTks5vMycjgaJpZM4a4A7H.

--
Vanessa McHale
Functional Compiler Engineer | Chicago, IL
Website: www.iohk.io http://iohk.io
Twitter: @vamchale
PGP Key ID: 4209B7B5
Input Output http://iohk.io
Twitter https://twitter.com/InputOutputHK Github
https://github.com/input-output-hk LinkedIn
https://www.linkedin.com/company/input-output-global
This e-mail and any file transmitted with it are confidential and
intended solely for the use of the recipient(s) to whom it is addressed.
Dissemination, distribution, and/or copying of the transmission by
anyone other than the intended recipient(s) is prohibited. If you have
received this transmission in error please notify IOHK immediately and
delete it from your system. E-mail transmissions cannot be guaranteed to
be secure or error free. We do not accept liability for any loss,
damage, or error arising from this transmission

Well, I think step one is a GDB-style CLI debugger. In terms of the UI I think implementing the Debug Adapter Protocol would be a good way to go.


Module parse failed in emotion/src/index.js (12:19) and emotion/src/utils.js (3:5), unexpected token

I'm getting the following error output from webpack when attempting to build a simple component that uses emotion:
WARNING in ./~/emotion/src/react/index.js
81:11-12 "export 'createElement' (imported as 'h') was not found in 'react'

ERROR in ./~/emotion/src/utils.js
Module parse failed: /mnt/c/Users/Granmoe/projects/boilerplates/react-redux-saga-boilerplate/node_modules/emotion/src/utils.js Unexpected token (3:5)
You may need an appropriate loader to handle this file type.
| // @flow weak
| export function omit(
|   obj: { [string]: any },
|   testFn: (key: string, obj: any) => boolean
| ) {
@ ./~/emotion/src/react/index.js 3:0-52
@ ./~/emotion/lib/react/index.js
@ ./~/emotion/react/index.js
@ ./src/components/app.jsx
@ ./src/main.jsx
@ multi (webpack)-dev-server/client?http://localhost:3000 webpack/hot/dev-server ./src/main.jsx

ERROR in ./~/emotion/src/index.js
Module parse failed: /mnt/c/Users/Granmoe/projects/boilerplates/react-redux-saga-boilerplate/node_modules/emotion/src/index.js Unexpected token (12:19)
You may need an appropriate loader to handle this file type.
| sheet.inject()
|
| export let inserted: { [string | number]: boolean | void } = {}
|
| export function flush() {
@ ./~/emotion/src/react/index.js 2:0-30 6:0-14:17
@ ./~/emotion/lib/react/index.js
@ ./~/emotion/react/index.js
@ ./src/components/app.jsx
@ ./src/main.jsx
@ multi (webpack)-dev-server/client?http://localhost:3000 webpack/hot/dev-server ./src/main.jsx

I read something in another issue about problems being caused by having your webpack entry file in a source dir named "src," which mine is, as you can see below.
I do have the emotion babel plugin in my babelrc:
 "env": {
   "production": {
     "presets": [["es2015", { "modules": false }], "react", "stage-1"],
     "plugins": ["emotion/babel"]
   },
   "test": {
     "presets": [
       ["env", {
         "targets": {
           "chrome": 60
         }
       }],
       "react",
       "stage-1"
     ],
     "plugins": ["emotion/babel", "babel-plugin-transform-es2015-parameters", "babel-plugin-transform-es2015-destructuring"]
   },
   "development": {
     "presets": [
       ["env", {
         "targets": {
           "chrome": 60
         }
       }],
       "react-hmre",
       "react",
       "stage-1"
     ],
     "plugins": ["emotion/babel", "babel-plugin-transform-es2015-parameters", "babel-plugin-transform-es2015-destructuring"]
   }
 }
}

And here's my webpack config:
  devtool: IS_DEV ? 'inline-source-map' : 'eval',
  entry: './src/main.jsx',
  output: {
    filename: 'bundle.js'
  },
  resolve: {
    modules: [
      'src',
      'node_modules'
    ],
    extensions: ['.json', '.js', '.jsx']
  },
  plugins, // uglify, image min, etc
  module: {
    rules: [
      {
        test: /\.jsx?$/,
        exclude: /node_modules/,
        loader: 'babel-loader'
      }, {
        test: /\.css$/,
        exclude: /node_modules/,
        loaders: [
          'style-loader',
          'css-loader'
        ]
      }
    ]
  },
  devServer: {
    historyApiFallback: true,
    port: 3000,
    compress: false,
    inline: true,
    hot: true,
    stats: {
      assets: true,
      children: false,
      chunks: false,
      hash: false,
      modules: false,
      publicPath: false,
      timings: true,
      version: false,
      warnings: true
    }
  }
}

Here's one of my components:
import React from 'react'
import { Fragment, Link } from 'redux-little-router'
import styled from 'emotion/react'

import Counter from 'components/counter.jsx'

const App = styled.div`
  margin: 30px;
`

const List = styled.List`
  list-style-type: none;
  display: flex;
  justify-content: space-around;
  margin-bottom: 15px;
`

const ListItem = styled.ListItem`
  display: inline;
`

export default () =>
  <App>
    <List>
      <ListItem><Link href="/react-redux-saga-boilerplate/counter">Async Counter</Link></ListItem>
      <ListItem><Link href="/react-redux-saga-boilerplate/foo">Foo</Link></ListItem>
      <ListItem><Link href="/react-redux-saga-boilerplate/bar">Bar</Link></ListItem>
    </List>

    <Fragment forRoute="/react-redux-saga-boilerplate/counter">
      <Counter />
    </Fragment>
    <Fragment forRoute="/react-redux-saga-boilerplate/foo">

Just a placeholder

    </Fragment>
    <Fragment forRoute="/react-redux-saga-boilerplate/bar">

Some content coListd go here

    </Fragment>
  </App>

emotion version: 7.0.13
react version: 15.6.1

Reproduction:
Run
./node_modules/webpack-dev-server/bin/webpack-dev-server.js
with BABEL_ENV and NODE_ENV === "development"

Closing as I got the fix in the slack channel. Fixed by changing
  resolve: {
    modules: [
      'src',
      'node_modules'
    ]
  }

to
  resolve: {
    modules: [
      path.resolve(__dirname, 'src'),
      'node_modules'
    ]
  }

For anyone curious why this happens: it happens because webpack tries to look for the react module so it goes up the file tree until it finds a module directory which in this case is src then node_modules so it looks in src and finds a react folder and uses the index.js file from that folder as the react module.
We're planning to switch to a monorepo with a seperate package for the react version soon which will completely fix this problem.


Fix entry points

From version 3.4.0 the entry point for sequener, macroexecutor and showscan changed on the setup.py. The sequencer is the only which fail, but the other use the wrong code.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Thanks for the fix! I had missed that.


spec is bit out of order

the specfile contains release twice 1.16.0-1:
* Mon Oct 09 2023 David Pascual Hernandez<EMAIL_ADDRESS>- 1.16.0-1
- e8e20f1 chore(ci): Fix release workflow build step checking out wrong commit (David Pascual)
- 97a7cd0 chore: Bump asyncopenstackclient dependency version (David Pascual)
- 41b12e7 chore: Release version 1.16.0 (github-actions)
- 278d1b1 chore(release): Add PyPI action & extract copr step (Tibor Dudlák)
- 9bbd987 chore: Bump python-semantic-release to v7.34.4 (Tibor Dudlák)
- d6b7298 feat: Add new dependecies to mrack.spec file (David Pascual)
- 7bbda34 feat(OpenStack): Add clouds.yaml as an authentication method (David Pascual)
- a5b32e3 feat(OpenStack): Import publick key on provision (David Pascual)
- 1a29d86 test: fix pylint issues and use isinstance (Tibor Dudlák)
- db74ae0 fix(Beaker): Exception has been thrown as raise missed argument (Tibor Dudlák)
- de027fa docs(Beaker): Add hostRequires documentation section to guides (David Pascual)

* Tue Sep 19 2023 Tibor Dudlák<EMAIL_ADDRESS>- 1.16.0-1
- 278d1b1 chore(release): Add PyPI action & extract copr step (Tibor Dudlák)
- 9bbd987 chore: Bump python-semantic-release to v7.34.4 (Tibor Dudlák)
- d6b7298 feat: Add new dependecies to mrack.spec file (David Pascual)
- 7bbda34 feat(OpenStack): Add clouds.yaml as an authentication method (David Pascual)
- a5b32e3 feat(OpenStack): Import publick key on provision (David Pascual)
- 1a29d86 test: fix pylint issues and use isinstance (Tibor Dudlák)
- db74ae0 fix(Beaker): Exception has been thrown as raise missed argument (Tibor Dudlák)
- de027fa docs(Beaker): Add hostRequires documentation section to guides (David Pascual)

may I kindly ask you @dav-pascual to see which section is correct and remove broken one?

there might be an issue when only fix commits are present and version is pre-loaded wrong way... i think this might be fixed please check but at least in changelog could you plase change version to 1.16.1 if even correct or.... just merge commit history in this section... I can try to check at evening...

thanks for reporting Tibor, will take as look as well :)

so there is no 1.16.1 .... sory for confusion

yes, this is what i meant

/packit propose-downstream

Thanks for fixing it in fedora


The new syntax for rgb is not supported

Extension Version: latest
VS Code Version: 1.61
OS Version: win10
Node.js Version (result of command node -v): 14.18.1
NPM Version (result of command npm -v): 7.23.0

Steps to Reproduce:

Related references: https://stylelint.io/user-guide/rules/list/color-function-notation/
Log (All lines):
(Click menu of VS Code "Help > Toggle Developer Tools", and click "Console" tab)

I'll close this issue now but still welcome comment.


Prevent infinite recursion on save

Prevent infinite recursion when saving a data containing a fragment which contains a fragment which contains a fragmentArray which has changed due to the way notifyPropertyChange works. setupFragment was calling setFragmentData which cause setupFragment to be called again, and never give it a change to assign fragment in internal model data.
Fixes a bunch of already reported but unsolved issues such as https://github.com/lytics/ember-data-model-fragments/issues/190 without any side effect. I did spend a lot of time to debug it, I'm sure a lot of people would like this to be merged.

Probably also fixes https://github.com/lytics/ember-data-model-fragments/pull/247

@JosephHalter Thanks a lot for this! As you noted, this has been a long running issue we've had trouble with. Would you mind creating a test for this infinite recursion scenario. It may be a little cumbersome, but it would really help us ensure it doesn't resurface.
Also, would you mind adding a code comment in the first if block (line ~100) that explains why the calls happen in a different order. Looking at the code it's unobvious to me.
Thanks again!

I try adding a test on it but it pass even without my patch. However it definitely fails in my application with Maximum call stack size exceeded. using exactly the same use case, a model with a fragment containing a fragment containing a fragmentArray which has changed. When _adapterDidCommit is called it calls setupFragment which calls setFragmentData which does content.replace(0, get(content, 'length'), processedData) which triggers notifyPropertyChange which once again come into setupFragment and since we didn't assign the fragment in _fragments yet it calls once again setFragmentData instead of just returning, resulting in an infinite loop where the assignation is never done. I saw plenty of references to this bug so it's not just me: https://github.com/emberjs/ember.js/pull/13957 Using previous versions of ember-data or ember-data-model-fragments did nothing to solve it either.
I suspect the ember runloop just doesn't work the same way when in test. I don't know how to make a better test. The fix has however no side effect. After more than 1 day of debug, adding logs and breakthrough points everywhere to pinpoint the exact bug.

Its ok with me to merge this with a comment explaining why the calls happen in different order

I added the comment and remove my attempt at adding a test on it since it didn't provide the expected value.


I'm curious, why don't you use ResNet in torchvision?

Is it because ResNet in torchvision is not up-to-date? Say, it may not have BN within the Residual Module?

The pre-trained backbone models are from torchvision.

But ... I see there's a version of your own here in the project's source code...you didn't use it?

I am not sure if the models from torchvison will change in the feature, so I also keep them by ourselves.

Thanks!